Skip to content
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

zombienet, retry only on 'runner_system_failure' #5882

Merged
merged 2 commits into from
Oct 2, 2024

Conversation

pepoviola
Copy link
Contributor

Only retry jobs on runner_system_failure.
Thx!

@pepoviola pepoviola requested a review from a team as a code owner October 1, 2024 08:49
@pepoviola pepoviola added R0-silent Changes should not be mentioned in any release notes T10-tests This PR/Issue is related to tests. labels Oct 1, 2024
@bkchr
Copy link
Member

bkchr commented Oct 1, 2024

@pepoviola can you explain what this brings us?

@pepoviola
Copy link
Contributor Author

pepoviola commented Oct 1, 2024

@pepoviola can you explain what this brings us?

Yes, the idea is to only retry automatically the jobs that fail at setup stage (e.g. the runner fail to be created) and do not retry in other stage, this will give us more visibility of fails in zombienet tests and we can act on them (fix/disable). Currently we are always retrying (2 times). I can adjust this settings if you think is better to have some retries.


Update: This also bump the version of zombienet (some fixes related to ci).
Thx!

@pepoviola pepoviola added this pull request to the merge queue Oct 2, 2024
Merged via the queue into master with commit 383180a Oct 2, 2024
217 checks passed
@pepoviola pepoviola deleted the jv-retry-only-on-setup-fails branch October 2, 2024 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
R0-silent Changes should not be mentioned in any release notes T10-tests This PR/Issue is related to tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants