Adding a Github Action that runs on CIRRUS runner scale sets and NCAR HPC containers#1071
Conversation
|
@mjs2369 Hey Marlee when you update this with main can you rebase rather than merge. Because we will probably want to squash commits testing the actions. Cheers, |
5def75b to
be49488
Compare
|
This is really cool, I think this can be a matrix of compiler, mpi |
|
Any update on why all the CIRRUS runners are failing? |
@hkershaw-brown yes, I just heard back from Ben Kirk. The cause is currently unknown, but is definitely on the CIRRUS-side and has been reported to the CIRRUS admins. There is currently no timeline for the fix, but I will run the final tests with the matrix as soon as I get the go-ahead. |
|
Note on feedback to GitHub on ❌ vs |
hkershaw-brown
left a comment
There was a problem hiding this comment.
Leaving this on 'request changes'.
Needs CIRRUS to be online 🤦♀️ to test
rebase (squash) commits into a single commit before merging.
|
Is CIRRUS still down? |
|
@hkershaw-brown yep, chatted with Ben at the conference too and there's currently no timeline. Should we put this on draft in the mean time? |
well, that is less than ideal. |
|
I'm rebasing this to be up-to-date with main, watch out when you pull. |
2e8d4dd to
57d95be
Compare
4d076a1 to
b1a2f7f
Compare
|
@mjs2369 can you rebase this so it is just a single commit (rather than two commits, code & merge commit with main). |
a968023 to
ac0ad8e
Compare
… with all mpi options and gfortran, nvhpc, intel-oneapi
|
@hkershaw-brown done |
hkershaw-brown
left a comment
There was a problem hiding this comment.
This is really nice Marlee, great work on all the container and cirrus interactions.
I'd like to bundle this with the next release, next week (when I am back in the office).
@hkershaw-brown merge this before #1069
Description:
Adds a Github Action that runs Lorenz_96 on CIRRUS runner scale sets and NCAR HPC containers. Runs with all mpi options (mpi, mpif08, nompi) with the compilers gfortran, and nvhpc, and intel-oneapi.
Note that this action will be run on every pull request but will NOT be required to pass to merge into main. @hkershaw-brown I do not have the permission to change these settings in the DART repo, if you could please ensure this is correctly set.
Fixes issue
No issue in for this, woops
Types of changes
Documentation changes needed?
Updated developer's guide documentation.
Tests
Please describe any tests you ran to verify your changes.
Tested action on push, all jobs succeeded.
Checklist for merging
Checklist for release
Testing Datasets