-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
fix: (ansible) - Errors in the event have not been re-trigging the reconcile. #2375
fix: (ansible) - Errors in the event have not been re-trigging the reconcile. #2375
Conversation
376395f
to
90bfa4e
Compare
90bfa4e
to
c2ab559
Compare
c2ab559
to
e60ced6
Compare
5142e28
to
465e68c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm, @shawn-hurley maybe you want to have a quick look if you have a sec
/retest Please review the full test history for this PR and help us cut down flakes. |
2 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
d5a45b8
to
1ff21bf
Compare
/hold cancel |
/lgtm |
Hi @shawn-hurley it was changed could you please give a look and see if it is ok for you now? |
/retest |
/retest Please review the full test history for this PR and help us cut down flakes. |
2 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
1ff21bf
to
53fc9fe
Compare
53fc9fe
to
7cdd8dd
Compare
c69b580
to
2a5b427
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
Description of the change:
Fix issue: #2372
Errors in the event have not been re-trigging the reconcile.
Motivation for the change:
Closes #2372
Notes
Local tests to verify it can be done with the image:
quay.io/camilamacedo86/ansible-operator:dev
The user POC in the #2372 was used to check.