-
Notifications
You must be signed in to change notification settings - Fork 15
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
ansible 2.0 support? #22
Comments
👍 :) |
Unfortunately I'm not using Ansible much anymore, but I'd be happy to take a look at PRs with 2.0 support. |
Are you using something else? Just curious what you decided to go with of you did. On Wed, Jan 27, 2016 at 9:25 AM -0800, "Dan" [email protected] wrote: Unfortunately I'm not using Ansible much anymore, but I'd be happy to take a look at PRs with 2.0 support. — |
@jkomara Sorry for the late response. I'm using puppet now. Though I have my beefs with Ansible, the primary reason I've changed is because my employer changed. |
#28 addresses this issue. when merged I'll close this. |
master branch has code to support Ansible 2.0. a v2.0.0 will follow in a couple of days. |
Awesome! Thank you.
From: Pedro Salgado [email protected] Closed #22. — |
There are still two users of ansible.runner - show_vars.py and show_template.py:
|
@opoplawski what ansible version are you using? our tests are passing in ansible 1.9.4 and 2.0.1.0
|
oh wait! don't know if the tests cover that code... |
FWIW - ansible 2.0.2.0. |
tests don't cover the case above : reference here. need to add tests for this and patch the code... |
working on this at issue-41 branch. |
Okay, I'll keep an eye on that then. Still see 4 test failures on that branch at the moment. |
yes, I'm still working on this... |
Hi @steenzout, are you still working on that ? |
@sylr this ain't a priority for me anymore. the current work on this is at this branch: https://github.com/dellis23/ansible-toolkit/tree/issue-41 feel free to PR. |
Needed the https://github.com/aioue/pilfer Needs |
Just came across this today and it looks awesome. I tried running it but it looks like it requires runner and the API was rewritten in 2.0, no longer using runner.
Are there any plans to support 2.0?
Thanks.
The text was updated successfully, but these errors were encountered: