Naor Livne

Is everything alright with the Jenkins builds? it fails on SwarmTest.test_init_swarm_with_ca_config with an error of ` docker.errors.APIError: 503 Server Error: Service Unavailable ("Swarm is encrypted and needs to be unlocked...

Sorry about the million changes requested, I see the direction your taking it but as it's a rather big refactor there are a lot of little bugs in it that...

I think you misunderstood. I agree that this change is needed just that the merge will happen after the syntax\unit-test\misc issues are resolved.

Python has a styling guide named PEP8 which is where the 79 line length comes from, I tend to prefer to sticking to the language best practice even if personally...

Your right about the pep8 bot so let's keep it at 119 for the time being & I'll have it changed to 79 project wide down the road

Rather then polling webhook reply once it's done will likely be a better way forward, can't believe I didn't think of it before, no need for a DB as it...

This will likely need to be combined with threading (as an option) so that by default everything work as is (no threads, wait for the response) but if threading is...

A temp directory won't work as it will mean that local state storage isn't possible to use, however this does seem like a problem that only happens if your using...

Issue seemed resolved with the gunicorn `sync` worker as I tried replicated your test & couldn't replicate the issue with it

For me personally the issue is with 0.15+ but as I prefer sticking to the latest version if https://github.com/beelit94/python-terraform/issues/116 will be merged and allow us to work with the latest...