-
Notifications
You must be signed in to change notification settings - Fork 55
[skip ci] Updates for spack v1 at NAS #1810
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
[skip ci] Updates for spack v1 at NAS #1810
Conversation
|
@climbfuji Many a change pushed based on your suggestions. Please take another look. |
repos/spack_stack/spack_repo/spack_stack/packages/gmao_swell_env/package.py
Outdated
Show resolved
Hide resolved
|
@mathomp4 Do you want to merge this as is and do the spack package updates in a separate PR? |
|
@climbfuji Okay. I think I did the packages one here: My first time doing something like this, so hopefully it's okay. :) Note: I didn't update the submodule pointer here, because I guess I thought that would be done after JCSDA/spack-packages#22 was merged? I might not have the order right. |
Description
This PR has updates for spack-stack v2 at NAS.
Beyond the updates for the machine itself, I'm also trying to close #1139 and build FMS in two ways with both GFS and GEOS constants.
At the moment, both variants do build, but weirdly I only get a suffix in the modulefiles for GFS→ This is no longer true. Was a typo on my end.The other issue with this is that I'm not sure how to handle the spack-stack unique modules. When I build this, the fms versions are all wonky. To be "safe" only the GEOS important ones should probably default to
constants=GEOSFMS, things likejedi-geos-envandgeos-gcm-env(which at the moment have internal FMS, but soon will not). I'm not sure how to make these default toconstants=GFS. I mean...some did...Dependencies
I will need to make a PR to spack-packages for some package updates I need. I'll work with @climbfuji or @AlexanderRichert-NOAA to teach me the right way to do this. I'm not the best at submodules
Issues addressed
Closes #1139
Applications affected
I guess
ufs-weather-model-env,ufs-srw-app-envandjedi-ufs-envare now picking up the wrong FMS. Nuts.Also, I updated
geos-gcm-envandjedi-geos-envto[email protected]because we use that now.Systems affected
NAS directly.
In trying to look at #1139, all other systems?
Testing
Checklist
doc/source/PreConfiguredSites.rstanddoc/source/MaintainersSection.rst