-
Notifications
You must be signed in to change notification settings - Fork 32
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 wcoss2.intel.lua to spack-stack #1435
Conversation
g-w CI has been successfully run on WCOSS2 (Cactus). All jobs in all tested g-w configurations completed without error. Details are found in issue #1350. The hera-GW-RT label will be applied to this PR as a sanity check. The changes in this PR should not impact the building or execution of GDASApp components on NOAA RHPDCS machines. This PR only impacts WCOSS2 (Cactus and Dogwood). |
Assuming hera-GW-RT passes, this PR will be marked Ready for review |
Automated GW-GDASApp Testing Results:
|
Automated GW-GDASApp Testing Results:
|
@DavidNew-NOAA : The failure of the g-w based gdasapp ctests is due to
For example,
|
Ticket RDHPCS#2025010854000354 opened with MSU help desk requesting Attempts to run g-w based GDASApp ctests from the Hercules For the time being we should not trigger g-w based GDASApp ctests for the following configurations
using the Developers belonging to the MSU FYI, g-w sets the default value for variable
|
Mark this PR as Ready for review because g-w CI passes on WCOSS2 (Cactus). The focus of this PR is updating the GDASApp build to WCOSS2 spack-stack. This works. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, I was able to build on WCOSS on Wednesday with this branch
WCOSS2 issues Two items to be aware of following the WCOSS2 upgrade:
This change impacts GDASApp executables. They no longer find GDASApp libraries. We need to add the GDASApp build path to
The following lines need to be added to g-w
It is also recommended that we add the following to g-w
|
Automated GW-GDASApp Testing Results:
|
Are we cleared to use spack-stack on WCOSS2 for regular development. |
@aerorahul : Understood. NCO and GDIT staff are aware that GDASApp is using this version of WCOSS2 spack-stack. A developer GSI fork contains a branch using the same WCOSS2 spack-stack. We've been testing the NCO implementation, identifying missing modules, and pointing out other issues. |
Description
This PR updates
wcoss2.intel.lua
to use spack-stack/1.6.0.Companion PRs
none
Issues
Resolves #1350
Resolves #1331
Resolves #1336
Resolves g-w #3100
Automated CI tests to run in Global Workflow