-
Notifications
You must be signed in to change notification settings - Fork 263
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
Revert "Ansible-runner: Fix getting status of playbook" and Revert "Set initial event number for async ansible commands" #496
Conversation
fa7fbde
to
49b9fbe
Compare
49b9fbe
to
3c101d7
Compare
/ost |
it doesn't make sense to blindly change things or revert, added more debugging now in #502 and once we see the issue again it should give us more insight into what's going on. Then we can see what fix or revert is needed |
I agree with Michal's comment above but I meant to elaborate within the commit message or/and the description of the PR so we'll be able to tell later on, if needed, why it was reverted |
/ost |
#546 fixed it good enough |
Reverts #468 and #499