-
Notifications
You must be signed in to change notification settings - Fork 186
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
[tests-only][ociswrapper] Fix oCIS restart when it exits unexpectedly during startup #7072
Conversation
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.
Seems OK. I suppose that the actual testing software will see the error (as a 500 status returned by the API, or...) and will be able to make the decision if it retries, or fails the test.
We do want to know when "bad things" go wrong and be able to see test failures. We don't want to actually end up hiding "intermittent nasty problems" by doing retries.
yeah, the ociswrapper only retries if the |
e4e14b9
to
73c8925
Compare
f67c6f9
to
44585e0
Compare
b93d66e
to
d1f57e2
Compare
this was not fully true. Now the recent commits fix this. |
a250543
to
d4be6a8
Compare
Kudos, SonarCloud Quality Gate passed! |
[tests-only][ociswrapper] Fix oCIS restart when it exits unexpectedly during startup
Description
Improves:
Related Issue
Part of #6850
How Has This Been Tested?
Types of changes
Checklist: