-
Notifications
You must be signed in to change notification settings - Fork 2
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
Incline CPU Testing + CI #25
Conversation
"description": "Custom changes specific for Deception" | ||
}, | ||
{ | ||
"name": "incline", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note we will probably have to have incline-cpu
vs incline-gpu
, and as @nkoukpaizan might want to use other compilers, we can also have ascent-llvm
vs ascent-gcc
and go from there? I think keeping this name tied to a spack config is key, and making the names simple and easy to use would be nice as well.
5d74d64
to
45bf639
Compare
Incline build and ctest work, but when trying
|
Started working on the module rebuild pipelines in the meantime... some perms issues for the service account runner: https://gitlab.pnnl.gov/exasgd/resolve-mirror/-/jobs/131472#L75 Ping Kevin Fox? Or we just make the resolve dir writeable for the exasgd group (like for exago ci)? |
That's my bad, @jaelynlitz. If you don't mind, please create issue and assign it to me. In the meantime, you can comment out |
I believe this issue is fixed in #34. |
5420344
to
b76d25e
Compare
Incline build and test pipelines pass after a rebase on develop with fixes from @ryandanehy and @pelesh! If we want to merge this progress, I can remove the code associated with the spack rebuild pipelines and tackle those in a separate PR. |
I think that would be a good break point. I suggest we merge this progress, first. |
Ready to merge |
Co-authored-by: Cameron Rutherford <cameron.rutherford@me.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will merge as soon as all tests pass.
From #23, need to add same CI config to Incline, and create issue for Spack modules on Incline in CI