Skip to content

Make Nightly Tests Continue on Error #2910

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

Merged
merged 1 commit into from
Feb 25, 2025

Conversation

AmirhosseinPoolad
Copy link
Contributor

Basically the same thing as #2905 but uses a different way to do it that hopefully avoids the issue of the run being considered successful even if some tests have failed. Did it in a different PR and branch because I didn't want to mess with the origin too much.

Currently, the Nightly Tests will stop whenever any of the Nightly Tests
fail (and will not run the following tests). So for example, if Nightly
Test 1 has a QoR failure, all other Nightly tests will not be run.

Made each step in the Nightly Test manual job continue on error. This
will run the following steps even if one of them fail.

Suggested-by: Alex Singer
@AmirhosseinPoolad
Copy link
Contributor Author

@AlexandreSinger Do you mind checking to see if this fixes the problem?

@github-actions github-actions bot added the infra Project Infrastructure label Feb 24, 2025
Copy link
Contributor

@AlexandreSinger AlexandreSinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This worked perfectly!
image

Very well done!

@AlexandreSinger AlexandreSinger merged commit ceae587 into master Feb 25, 2025
36 of 37 checks passed
@AlexandreSinger AlexandreSinger deleted the nightly-test-fail-nostop branch February 25, 2025 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infra Project Infrastructure
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants