-
Notifications
You must be signed in to change notification settings - Fork 59
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
Breaks with ansible devel/ansible 2.8+ #28
Comments
🤔 something to consider |
Considered. This describes the problem since the dawn of time (or at least ansible time). pytest-ansible relies on potentially unstable APIs (there are no stable ones). There are ways we can improve the situation, and Let's leave this issue open until pytest-ansible can adapt to the changes. |
Fixed by #30 |
Earlier i was seeing this issue and then I updated my pytest-ansible and ansible , Command: . . <10.0.151.209> SSH: EXEC ssh -vvv -C -o ControlMaster=auto -o ControlPersist=60s -o KbdInteractiveAuthentication=no -o PreferredAuthentications=gssapi-with-mic,gssapi-keyex,hostbased,publickey -o PasswordAuthentication=no -o 'User="root"' -o ConnectTimeout=10 -o ControlPath=/root/.ansible/cp/af01c08f47 10.0.151.209 '/bin/sh -c '"'"'chmod u+x /root/.ansible/tmp/ansible-tmp-1596797381.57-70538618513074/ /root/.ansible/tmp/ansible-tmp-1596797381.57-70538618513074/AnsiballZ_command.py && sleep 0'"'"'' Versions used: |
I get this traceback:
I'm guessing ansible/ansible@7e92ff8
impacted this?
@abadger if you have any input on what proper use should be in these two places, or how to be compatible with older versions as well as the new way, it would be appreciated!
pytest-ansible/pytest_ansible/module_dispatcher/v2.py
Line 80 in a3a1418
pytest-ansible/pytest_ansible/module_dispatcher/v24.py
Line 81 in a3a1418
Maybe we need a
module_dispatcher/v28.py
The text was updated successfully, but these errors were encountered: