-
Notifications
You must be signed in to change notification settings - Fork 45
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes for Ansible 2.0 #38
base: master
Are you sure you want to change the base?
Conversation
Main Problem ist: I'm not yet sure if I can incorporate the changes in a way to support both, ansible 2.0 and 1.9. Since we're using this deployment for our production system here, and are not able to update 2.0 for a while, this is somewhat... challenging ;-) |
@maschmann I tried to make sure it was backwards compatible! There are a few more changes for things I don't use in my own production I believe, also let me update the changes because |
Either way you need to get it working for 2.0 now it's the main release ;) Let me know if I can help! |
Hi, |
I've had a look into the include changes: It is possible to do this in the way you've implemented id, but just checking the included task variable to be not empty seems to be a bad idea, since it breaks if e.g. the null value handling of ansible changes. Would be better to use a file stat instead? //edit |
No description provided.