This should avoid a race where the kill is not done when spinning up the new container. podman stop waits 10 seconds by default.
Fixes #27777 (review)
ci: Use podman stop over podman kill #27844
pull maflcko wants to merge 1 commits into bitcoin:master from maflcko:2306-ci-pod-stop- changing 1 files +1 −1-
maflcko commented at 2:59 PM on June 9, 2023: member
-
faaa62754e
ci: Use podman stop over podman kill
This should avoid a race where the kill is not done when spinning up the new container. podman stop waits 10 seconds by default.
-
DrahtBot commented at 2:59 PM on June 9, 2023: contributor
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--021abf342d371248e50ceaed478a90ca-->
Reviews
See the guideline for information on the review process.
Type Reviewers ACK dimitaracev If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
<!--174a7506f384e20aa4161008e828411d-->
Conflicts
Reviewers, this pull request conflicts with the following ones:
- #27793 (ci: label docker images and prune dangling images selectively by stickies-v)
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.
- DrahtBot added the label Tests on Jun 9, 2023
-
dimitaracev commented at 6:42 PM on June 10, 2023: contributor
ACK faaa627
- fanquake merged this on Jun 12, 2023
- fanquake closed this on Jun 12, 2023
- maflcko deleted the branch on Jun 12, 2023
- sidhujag referenced this in commit 0c2423005b on Jun 12, 2023
- fanquake referenced this in commit 6f7a0ae58b on Jun 15, 2023
- fanquake referenced this in commit 642b5dd1b4 on Jun 16, 2023
- fanquake referenced this in commit e444610662 on Sep 26, 2023
- fanquake referenced this in commit defdc15023 on Sep 26, 2023
-
- fanquake referenced this in commit 1416d09cba on Oct 6, 2023
- bitcoin locked this on Sep 25, 2024
