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

nightly regression testing causes disk quota error #764

Closed
BenWibking opened this issue Oct 1, 2024 · 1 comment
Closed

nightly regression testing causes disk quota error #764

BenWibking opened this issue Oct 1, 2024 · 1 comment
Labels
bug: wrong answer/failure/crash Something isn't working CI priority:high high priority

Comments

@BenWibking
Copy link
Collaborator

Describe the bug
We run out of disk space on avatargpu, since the nightly regression tests leave their plotfiles in my home directory. This causes all the GPU tests to fail.

To Reproduce
Steps to reproduce the behavior:

  1. Let the regression tests run for a few weeks.
  2. Observe the GPU CI tests to fail with a disk quota error.

Additional context
This should be fixed using this option in the regression testing framework: AMReX-Codes/regression_testing#146

@BenWibking BenWibking added bug: wrong answer/failure/crash Something isn't working CI priority:high high priority labels Oct 1, 2024
github-merge-queue bot pushed a commit that referenced this issue Oct 4, 2024
### Description
This runs the regression testing Python script with the
`--clean_testdir` option, which deletes the test output directory after
a given test passes.

### Related issues
#764

### 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:_
- [x] I have added a description (see above).
- [x] I have added a link to any related issues see (see above).
- [x] I have read the [Contributing
Guide](https://github.com/quokka-astro/quokka/blob/development/CONTRIBUTING.md).
- [ ] 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.
@BenWibking
Copy link
Collaborator Author

Fixed by #769

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug: wrong answer/failure/crash Something isn't working CI priority:high high priority
Projects
None yet
Development

No branches or pull requests

1 participant