Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Upload results even if regression tests fail #711

Merged

Conversation

BenWibking
Copy link
Collaborator

@BenWibking BenWibking commented Aug 9, 2024

Description

Upload the test results even if a test fails. Currently, the pipeline is canceled if any of the tests fail, and the results aren't available on the website (https://quokka-astro.github.io/regression-testing-results/).

Documentation: https://learn.microsoft.com/en-us/azure/devops/pipelines/process/conditions?view=azure-devops

Related issues

N/A

Checklist

Before this pull request can be reviewed, all of these tasks should be completed. Denote completed tasks with an x inside the square brackets [ ] in the Markdown source below:

  • I have added a description (see above).
  • I have added a link to any related issues see (see above).
  • I have read the Contributing Guide.
  • I have added tests for any new physics that this PR adds to the code.
  • I have tested this PR on my local computer and all tests pass.
  • I have manually triggered the GPU tests with the magic comment /azp run.
  • I have requested a reviewer for this PR.

Upload the test results even if a test fails. Currently, the pipeline is canceled if any of the tests fail, and the results aren't available on the website (https://quokka-astro.github.io/regression-testing-results/).
@BenWibking BenWibking requested a review from markkrumholz August 9, 2024 02:57
@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Aug 9, 2024
@BenWibking BenWibking enabled auto-merge August 9, 2024 02:57
@dosubot dosubot bot added the CI label Aug 9, 2024
@BenWibking BenWibking changed the title Update regression test pipeline Upload results even if regression tests fail Aug 9, 2024
@BenWibking BenWibking added this pull request to the merge queue Aug 9, 2024
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 9, 2024
Merged via the queue into development with commit bea14af Aug 9, 2024
19 checks passed
@BenWibking BenWibking deleted the BenWibking/upload-regression-results-always branch August 10, 2024 00:51
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
CI lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants