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

Preserve CI artifacts #22

Merged
merged 15 commits into from
Dec 3, 2024
Merged

Preserve CI artifacts #22

merged 15 commits into from
Dec 3, 2024

Conversation

brryan
Copy link
Collaborator

@brryan brryan commented Dec 2, 2024

Background

Right now, because the Darwin CI runs in a temporary directory, all artifacts are lost.

This PR sends artifacts to /usr/projects/jovian so all LANL people with Darwin access and jovian group membership can see them.

Fixes #16

Description of Changes

  • Create a /usr/projects/jovian/ci/[moniker]/pr[pr number] directory and use that for output in Darwin CI script
  • Update permissions of artifacts for jovian group access
  • Move SLURM output to artifacts directory

Checklist

  • New features are documented
  • Tests added for bug fixes and new features
  • (@lanl.gov employees) Update copyright on changed files

@brryan
Copy link
Collaborator Author

brryan commented Dec 2, 2024

Just one more issue I'll fix before merge -- right now we save all the data output files by default, which ends up being 734 MB per CI run in /usr/projects/jovian which we can't afford. I'll add an option to remove the data files after they're created.

tst/run_tests.py Outdated Show resolved Hide resolved
@brryan brryan merged commit 8aa03a9 into develop Dec 3, 2024
4 checks passed
@brryan brryan deleted the brryan/ci_artifacts branch December 3, 2024 21:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Have darwin CI store artifacts locally
3 participants