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

Add/fix build capability for Gaea-C5, Gaea-C6, and container #146

Merged
merged 3 commits into from
Nov 6, 2024

Conversation

DavidBurrows-NCO
Copy link
Contributor

After the recent Gaea-C5 OS upgrade, gsi-monitor fails to build.
This issues corrects Gaea-C5 build, adds Gaea-C6 build capability (following the ufs-wx-model PR2448), and adds containerized build capability.

Refs NOAA-EMC/global-workflow 3011
Refs NOAA-EMC/global-workflow 3025
Closes #145

@CoryMartin-NOAA
Copy link

Who has review/merge permission here? @EdwardSafford-NOAA , others? @EdwardSafford-NOAA if so, can you review?

Copy link
Collaborator

@EdwardSafford-NOAA EdwardSafford-NOAA left a comment

Choose a reason for hiding this comment

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

Looks OK to me.

@EdwardSafford-NOAA
Copy link
Collaborator

@CoryMartin-NOAA I do have review/merge access to this repo. I don't have access to gaea* so I can't test the build but I'll take it on faith that it works. I'm good with merging now unless you want a second set of eyes on this.

@CoryMartin-NOAA
Copy link

no I think it's fine I just saw that @DavidBurrows-NCO had several PRs open and I was doing my part to try to shepherd them along

@EdwardSafford-NOAA EdwardSafford-NOAA merged commit 879f590 into NOAA-EMC:develop Nov 6, 2024
4 checks passed
@DavidBurrows-NCO DavidBurrows-NCO deleted the feature/c5c6conbuild branch November 7, 2024 15:38
@DavidBurrows-NCO
Copy link
Contributor Author

Thanks you @CoryMartin-NOAA @EdwardSafford-NOAA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add/fix build capability for Gaea-C5, Gaea-C6, and container
4 participants