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.

Sign up for GitHub

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

Jump to bottom

Update scripts to build on both Gaea-C5 and C6 #995

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

Update scripts to build on both Gaea-C5 and C6 #995

wants to merge 7 commits into from

Conversation

DavidBurrows-NCO
Copy link
Contributor

@DavidBurrows-NCO DavidBurrows-NCO commented Oct 21, 2024 •
edited by GeorgeGayno-NOAA
Loading

DESCRIPTION OF CHANGES:

The Gaea-C5 OS upgrade changed the version numbers for stack-intel and stack-cray-mpich. This PR updates the module files to point to the new version numbers.

In addition, following the ufs-wx-model PR2448 for Gaea-C6 upgrade, target is set to either gaeac5 and gaeac6, and separate module files are created for each.

TESTS CONDUCTED:

Cloned and built on Gaea-C5 and C6.

Standard tests

  • Compile branch on all Tier 1 machines using Intel. Done on WCOSS2, Jet, Hera, Orion and Hercules using c1e23fd.
  • Test the link_fixdirs.sh script on all Tier 1 machines. Done on WCOSS2, Jet, Hera, Orion and Hercules using c1e23fd.

DEPENDENCIES:

NA

DOCUMENTATION:

NA

ISSUE:

References NOAA-EMC/global-workflow issue #3011

Resolves #994

@DavidBurrows-NCO
Copy link
Contributor Author

Hello @GeorgeGayno-NOAA I have included updates for building ufs-utils in a container provided by @mark-a-potts. This is work being coordinated with @aerorahul (@WalterKolczynski-NOAA added since Rahul is on leave) and references NOAA-EMC/global-workflow Issue #3025.
For links to the fix directories, we don't want to have the fix directories in the container so did not update link_fixdirs.sh.
Please let me know if including the container build is okay, and if so, how we should handle link_fixdirs.sh. Maybe a warning message?
Thanks!

@DavidBurrows-NCO DavidBurrows-NCO mentioned this pull request Oct 30, 2024
Open
10 tasks
GeorgeGayno-NOAA
GeorgeGayno-NOAA requested changes Nov 5, 2024
View reviewed changes
Copy link
Collaborator

@GeorgeGayno-NOAA GeorgeGayno-NOAA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@DavidBurrows-NCO - I had already approved this PR, but I see you are making more changes. I will re-review when you are done.

@DavidBurrows-NCO
Copy link
Contributor Author

@GeorgeGayno-NOAA Sorry about that! Code is finalized and is consistent with other global-workflow submodules. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GeorgeGayno-NOAA GeorgeGayno-NOAA GeorgeGayno-NOAA requested changes

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

ufs_utils fails to build on Gaea-C5 after OS upgrade
3 participants
@DavidBurrows-NCO @GeorgeGayno-NOAA @mark-a-potts