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

Update module file after Gaea-C5 OS upgrade #1070

Merged
merged 5 commits into from
Oct 21, 2024

Conversation

DavidBurrows-NCO
Copy link
Contributor

@DavidBurrows-NCO DavidBurrows-NCO commented Oct 17, 2024

The EPIC SI team has reinstalled spack-stack-1.6.0 at the same location it existed before. This PR updates version numbers in modulefiles/gaea.lua:
stack-intel 2023.1.0 to 2023.2.0
stack-cray-mpich 8.1.25 to 8.1.28

Resolves #1071
Refs NOAA-EMC/global-workflow #3011

@WenMeng-NOAA
Copy link
Collaborator

@DavidBurrows-NCO Please open a new issue linking to this PR. Are you working at EMC, EPIC or another affiliate?

@DavidBurrows-NCO
Copy link
Contributor Author

@WenMeng-NOAA I added the issue and global-workflow reference to the original PR comment. My main affiliation is EPIC, but this is work done alongside @aerorahul's global-workflow team.

@DavidBurrows-NCO
Copy link
Contributor Author

@WenMeng-NOAA @aerorahul Since the ufs-wx-model (Ref #2448) is setting up the gaeac5 and gaeac6 convention, should I go ahead and set that up for upp and other global-workflow submodules, i.e., convert all current gaea references to gaeac5. Follow up PRs can handle the C6 build/runs.

@WenMeng-NOAA WenMeng-NOAA added EPIC Support Requested Ready for Review This PR is ready for code review. labels Oct 18, 2024
@WenMeng-NOAA
Copy link
Collaborator

@WenMeng-NOAA @aerorahul Since the ufs-wx-model (Ref #2448) is setting up the gaeac5 and gaeac6 convention, should I go ahead and set that up for upp and other global-workflow submodules, i.e., convert all current gaea references to gaeac5. Follow up PRs can handle the C6 build/runs.

@DavidBurrows-NCO I would suggest adding modulefile updates for both Gaea C5 and C6 in UPP, so they will be propagate to UFSWM with the upp submodule upgrade.

@DavidBurrows-NCO
Copy link
Contributor Author

@WenMeng-NOAA I have updated this PR to include building on C6 and differentiated detect machine and modulefiles between gaeac5 and gaeac6. I re-cloned and tested builds on both systems. Let me know your thoughts. Thanks.

@WenMeng-NOAA
Copy link
Collaborator

@DavidBurrows-NCO I don't have Gaea access. Currently, the UPP RTs don't include Gaea. As long as you have completed the UPP executable build testing, it is fine with me.

@WenMeng-NOAA WenMeng-NOAA added Ready for commit queue No Baseline Change No baseline of the UPP regression tests are made. labels Oct 21, 2024
@WenMeng-NOAA
Copy link
Collaborator

The UPP RTs have been completed on WCOSS2 with no baseline changes.

@WenMeng-NOAA
Copy link
Collaborator

This PR is ready for merging.

@WenMeng-NOAA WenMeng-NOAA merged commit 4e41404 into NOAA-EMC:develop Oct 21, 2024
3 checks passed
@DavidBurrows-NCO
Copy link
Contributor Author

Thanks for your help @WenMeng-NOAA

@DavidBurrows-NCO DavidBurrows-NCO deleted the c5OSup branch October 21, 2024 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EPIC Support Requested No Baseline Change No baseline of the UPP regression tests are made. Ready for commit queue Ready for Review This PR is ready for code review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

UPP fails to build on Gaea-C5 after OS upgrade
2 participants