-
-
Notifications
You must be signed in to change notification settings - Fork 144
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 idempotency for RestartPolicy when MaximumRetryCount > 0 #607
Fix idempotency for RestartPolicy when MaximumRetryCount > 0 #607
Conversation
Build failed (check pipeline). For information on how to proceed, see https://review.rdoproject.org/zuul/buildset/222ddb6d1832469281a0af991a02202e ❌ tripleo-ci-centos-9-standalone FAILURE in 9m 35s |
I assume, my change does not cause this failure. Tell me, if I'm wrong. |
@sedrubal please commit with |
check-github |
Build failed (check pipeline). For information on how to proceed, see https://review.rdoproject.org/zuul/buildset/32d9640b359a46c3b724b687c19406ad ❌ tripleo-ci-centos-9-ovb-3ctlr_1comp-featureset001 NODE_FAILURE Node request 200-0006241071 failed in 0s |
Before this change, this task: ```ansible - containers.podman.podman_container: # ... restart_policy: on-failure:3 # ... ``` always caused this diff: ```diff --- before +++ after @@ -1 +1 @@ -restart_policy - on-failure +restart_policy - on-failure:3 ``` Signed-off-by: Sebastian Endres <[email protected]>
7b972ef
to
1da2e24
Compare
Build failed (check pipeline). For information on how to proceed, see https://review.rdoproject.org/zuul/buildset/bd7e913e724f4aaeb1bf284cc8c8b203 ❌ tripleo-ci-centos-9-standalone NODE_FAILURE Node request 200-0006241448 failed in 0s |
Thanks for merging. Sorry that I didn't provide a test, but it would cost me to much effort to dig into the tests... |
Before this change, this task:
always caused this diff: