Skip to content

Feature/add metplus capabilities noaacloud#375

Closed
EdwardSnyder-NOAA wants to merge 2 commits into
ufs-community:developfrom
EdwardSnyder-NOAA:feature/add-metplus-capabilities-noaacloud
Closed

Feature/add metplus capabilities noaacloud#375
EdwardSnyder-NOAA wants to merge 2 commits into
ufs-community:developfrom
EdwardSnyder-NOAA:feature/add-metplus-capabilities-noaacloud

Conversation

@EdwardSnyder-NOAA
Copy link
Copy Markdown
Collaborator

@EdwardSnyder-NOAA EdwardSnyder-NOAA commented Sep 23, 2022

DESCRIPTION OF CHANGES:

Added MET/METplus variables to the noaacloud machine file so we can run MET_verification E2E test on AWS. Also, modified the noaacloud's task local files to reflect a new spack stack location.

Ran MET_verification E2E test on AWS. Will run the fundamental tests through Jenkins since the local files were modified.

Type of change

  • 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

TESTS CONDUCTED:

  • hera.intel
  • orion.intel
  • cheyenne.intel
  • cheyenne.gnu
  • gaea.intel
  • jet.intel
  • wcoss2.intel
  • NOAA Cloud (indicate which platform)
  • Ran MET_verification test on AWS
  • Jenkins
  • fundamental test suite
  • comprehensive tests (specify which if a subset was used)

DEPENDENCIES:

DOCUMENTATION:

ISSUE:

This PR will help validate the MET/METplus installations on the noaa cloud platforms, which is captured in Issue 387.

CHECKLIST

  • My code follows the style guidelines in the Contributor's Guide
  • I have performed a self-review of my own code using the Code Reviewer's Guide
  • I have commented my code, particularly in hard-to-understand areas
  • My changes need updates to the documentation. I have made corresponding changes to the documentation
  • My changes do not require updates to the documentation (explain).
  • My changes generate no new warnings
  • New and existing tests pass with my changes
  • Any dependent changes have been merged and published

LABELS (optional):

A Code Manager needs to add the following labels to this PR:

  • Work In Progress
  • bug
  • enhancement
  • documentation
  • release
  • high priority
  • run_ci
  • run_we2e_fundamental_tests
  • run_we2e_comprehensive_tests
  • Needs Cheyenne test
  • Needs Jet test
  • Needs Hera test
  • Needs Orion test
  • help wanted

CONTRIBUTORS (optional):

@EdwardSnyder-NOAA
Copy link
Copy Markdown
Collaborator Author

Closing this PR as these changes have been incorporated in previous PRs (#388 and #413).

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.

1 participant