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

Nersc ci #600

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Nersc ci #600

wants to merge 3 commits into from

Conversation

stuartmcalpine
Copy link

Added a github actions workflow (nersc-ci.yml):

This triggers the NERSC CI pipeline. At the moment this is just triggered manually, however you can change this to on push, schedule, etc.

Added quick catalog test for ci (./tests/test_catalogs_ci-py):

This gets run with the --cori-ci flag for pytest. It just reads a few value from the cosmoDC2_v1.1.4_small, however you can change this to what you need.

Added a gitlab workflow (.gitlab-ci.yml):

This is what gets run on Cori, at the moment it just installs gcr-catalogs and runs pytest on the new test file

I don't have permission to run workflows on this repo (I think). The repos the nersc-ci.yml file points to are my NERSC GitLab repos. You'll have to make repos in your NERSC instance and change the values in the nersc-ci.yml file. Setup for repos is in the docs for CI repo (https://desc-continuous-integration.readthedocs.io/en/latest/desc/ci_at_nersc.html)

@stuartmcalpine
Copy link
Author

The CI now gets done on Cori and reported back here (see the "NERSC" status above).

The files are ready to be merged in, but the branch name in the workflow will have to be changed from nersc_ci to master. And the NERSC repos will have to be changed to your ones.

You may also want to add any additional tests to the test_catalogs_ci.py file.

@stuartmcalpine stuartmcalpine requested a review from yymao September 3, 2022 02:56
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.

1 participant