Skip to content

Update cloud to spack-stack 1.9.2 with GNU stack fix#902

Merged
RussTreadon-NOAA merged 46 commits into
NOAA-EMC:developfrom
TaylorRoper-NOAA:update/cloud_build
Jul 28, 2025
Merged

Update cloud to spack-stack 1.9.2 with GNU stack fix#902
RussTreadon-NOAA merged 46 commits into
NOAA-EMC:developfrom
TaylorRoper-NOAA:update/cloud_build

Conversation

@TaylorRoper-NOAA
Copy link
Copy Markdown
Contributor

Description

This PR removes the unloading of gnu from the cloud module file. Loading gnu adds GCC libraries and tools to the necessary paths that are needed in the stack. There are different GCC version defaults on the legacy versus latest NOAA RDHPCS cloud /apps snapshots (13.2.0 for legacy, 14.2.0 for latest), and these different version module files have different configurations so that the GCC version is specified as 13.2.0 to have consistency across the different /apps mounts.

Resolves #901

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

I cloned and ran the build script on PW AWS with latest /apps snapshot mounted. I am not currently able to test with the legacy /apps snapshot, but the GCC default version is specified as 13.2.0 in this PR which is the default GCC version on the legacy /apps snapshot so that the build should finish successfully regardless of which /apps snapshot is mounted.

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • New and existing tests pass with my changes
  • Any dependent changes have been merged and published

DavidHuber-NOAA and others added 30 commits June 3, 2025 14:44
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Comment thread modulefiles/gsi_noaacloud.intel.lua Outdated
@TaylorRoper-NOAA
Copy link
Copy Markdown
Contributor Author

Putting this PR into draft mode to include spack-stack 1.9.1 update.

@TaylorRoper-NOAA TaylorRoper-NOAA marked this pull request as draft July 3, 2025 15:27
@TaylorRoper-NOAA TaylorRoper-NOAA changed the title Add gcc version and maintain gnu stack on cloud Update to spack-stack 1.9.1 with GNU stack fix Jul 3, 2025
@TaylorRoper-NOAA
Copy link
Copy Markdown
Contributor Author

This PR is waiting for #890 to merge.

@TaylorRoper-NOAA TaylorRoper-NOAA changed the title Update to spack-stack 1.9.1 with GNU stack fix Update cloud to spack-stack 1.9.1 with GNU stack fix Jul 3, 2025
@TaylorRoper-NOAA TaylorRoper-NOAA marked this pull request as ready for review July 10, 2025 17:29
@TaylorRoper-NOAA
Copy link
Copy Markdown
Contributor Author

Marking this PR as ready for review. I've updated the cloud module files to use spack-stack 1.9.2 and tested the build after the merging of #890. The build is successful with no errors and one warning (deprecation):

/contrib/dev/GSI/src/gsi/stpjcmod.f90(920): warning #9111: The minus (-) operator for OpenMP* reduction clauses is deprecated.
!$omp parallel do private(k,j,i,ii) firstprivate(rcon,con) reduction(-:dmn) reduction(-:dmn2) collapse(2)

@DavidHuber-NOAA @weihuang-jedi This PR is now ready for review.

@TaylorRoper-NOAA TaylorRoper-NOAA changed the title Update cloud to spack-stack 1.9.1 with GNU stack fix Update cloud to spack-stack 1.9.2 with GNU stack fix Jul 14, 2025
Copy link
Copy Markdown
Collaborator

@DavidHuber-NOAA DavidHuber-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 good to me. Thanks Taylor!

@RussTreadon-NOAA RussTreadon-NOAA self-requested a review July 28, 2025 11:48
Copy link
Copy Markdown
Contributor

@RussTreadon-NOAA RussTreadon-NOAA left a comment

Choose a reason for hiding this comment

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

This PR only modifies modulefiles/gsi_noaacloud.intel.lua. Thus, this PR does not impact the machines which most developers use: Gaea C6, Hera, Hercules, Orion, Ursa, and WCOSS2.

I do not have cloud access to test the changes to modulefiles/gsi_noaacloud.intel.lua.

Approve pending confirmation that the changes work as intended.

@RussTreadon-NOAA
Copy link
Copy Markdown
Contributor

@weihuang-jedi : Do you have any input regarding this PR? We can schedule this PR for merger into GSI develop assuming you have no request for change or objections.

@TaylorRoper-NOAA
Copy link
Copy Markdown
Contributor Author

TaylorRoper-NOAA commented Jul 28, 2025

This PR only modifies modulefiles/gsi_noaacloud.intel.lua. Thus, this PR does not impact the machines which most developers use: Gaea C6, Hera, Hercules, Orion, Ursa, and WCOSS2.

I do not have cloud access to test the changes to modulefiles/gsi_noaacloud.intel.lua.

Approve pending confirmation that the changes work as intended.

Thank you, @RussTreadon-NOAA, for taking a look at this PR and moving it forward. I can confirm that these changes allow a successful GSI build on the cloud using spack-stack 1.9.2; see build logs on Hera/Ursa here: /scratch3/BMC/qosap/Taylor.Roper/shared/build_gsi.log. I have not tested the run-time functionality of GSI since GSI is only partially ported to the cloud at this point.

@weihuang-jedi
Copy link
Copy Markdown
Contributor

I have looked the code changes, which looks good to me.
I checked out the code with the code changes on AWS, and it compiles fine.

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.

Update cloud build env

4 participants