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
Retry hook scripts that use the exit code 142 #4718
Conversation
5a88c37
to
eca2402
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.
I ran count_fail_ratio make test KEEP_DB=1 TESTS=t/10-jobs.t
and got 20/20 stable runs.
EDIT: There seems to be a slight runtime increase of t/10-jobs.t. From 20 runs in origin/master the mean runtime is 10.1s, for your change it's 10.35s so an increase of 0.2s. Well, nevermind :)
Codecov Report
@@ Coverage Diff @@
## master #4718 +/- ##
=======================================
Coverage 98.07% 98.08%
=======================================
Files 374 375 +1
Lines 34673 34752 +79
=======================================
+ Hits 34007 34086 +79
Misses 666 666
Continue to review full report at Codecov.
|
Shouldn't be much of a runtime increase, since the only delay tests that actually run multiple times use a |
This patch turns the previous proof of concept into a proper feature for making hook scripts restartable with a special exit code. The hook script now runs in a separate Minion job, and all parameters are configurable with environment variables and job settings. Progress: https://progress.opensuse.org/issues/112523
eca2402
to
106beeb
Compare
commit 795d401 Merge: ef96c27 106beeb Author: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> AuthorDate: Thu Jun 23 21:10:16 2022 +0000 Commit: GitHub <noreply@github.com> CommitDate: Thu Jun 23 21:10:16 2022 +0000 Merge pull request os-autoinst#4718 from os-autoinst/k/restart_hook_scripts Retry hook scripts that use the exit code 142
This patch turns the previous proof of concept into a proper feature for
making hook scripts restartable with a special exit code. The hook
script now runs in a separate Minion job, and all parameters are
configurable with environment variables and job settings.
Progress: https://progress.opensuse.org/issues/112523