-
Notifications
You must be signed in to change notification settings - Fork 119
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
[develop] Update SRW with spack-stack version 1.5.0 (from 1.4.1) #969
Conversation
Remove depreciated variabled from met+ params
@MichaelLueken , @ulmononian caught one thing. We forget to change file system on build and workflow scripts for Gaea-C5 from F2 to F5. I just committed these two changes. Do you think there's any other place (like in Jenkins)? |
@RatkoVasic-NOAA - While queuing up the Jenkins tests this morning, it has come to my attention that Gaea C5 is no longer using the |
@MichaelLueken I found two more files pointing to old file system. |
@RatkoVasic-NOAA - There are no longer any nodes associated with |
@MichaelLueken I'm looking into UFS WM PRs, and they are changing Gaea's name from gaea-c5 to just gaea. Can you check with whomever changed by just taking off hyphen if they also agree on having just gaea? It would be great to have same name across all applications. |
@jkbk2004 @zach1221 do you know how this call was made for the ufs-wm? |
@RatkoVasic-NOAA - I'm using the |
@RatkoVasic-NOAA and @ulmononian - Talking with Zach, the change in PR #2115 is only for manual runs of the UFS WM regression tests. The Jenkinsfile is still pointing to gaeac5 for the UFS WM Jenkins tests. |
Additional information from Kris Booker:
So, for the purposes of the Jenkinsfile, renaming the |
@RatkoVasic-NOAA and @natalie-perlin - I have made the necessary modifications to allow the SRW App to successfully build on Gaea C5, but while attempting to run the WE2E coverage tests, the tests are all failing in make_grid with the following error message:
I'll continue to dig around and see what might be happening, but I would appreciate any assistance you can provide, especially if this error message was encountered on other machines transitioning to spack-stack v.1.50. My forked branch of @RatkoVasic-NOAA's ss150 branch can be found https://github.com/MichaelLueken/ufs-srweather-app/tree/ss150 |
@natalie-perlin - Making the changes to |
@RatkoVasic-NOAA - There were four WE2E comprehensive tests that failed on Jet:
The Jenkins working directory on Jet is /mnt/lfs1/NAGAPE/epic/role.epic/jenkins/workspace/fs-srweather-app_pipeline_PR-969/jet/expt_dirs. The The The The I'm resubmitting the failed jobs now and will let you know how they look. |
@RatkoVasic-NOAA - Three of the tests that had failed are now successfully passing on Jet:
The
It doesn't make sense to me that a single test is encountering a CFL violation and not the rest of the tests that also use For Gaea C5, running the WE2E fundamental tests on the machine fail in
This is using the |
@MichaelLueken I'm running now 'specify_template_filenames' test on Jet. Last time I ran it it worked. I'll start from beginning. |
@RatkoVasic-NOAA - After many
|
On Gaea C5, while compiling, I am seeing the following messages, which didn't appear before the transition to the F5 filesystem:
Additionally, I'm trying to see if the modifications to It's unclear why the |
Remove gaea-C4
Comment out gaeac5 in Jenkinsfile
DESCRIPTION OF CHANGES:
Update SRW with spack-stack 1.5.0
Machines affected:
Type of change
TESTS CONDUCTED:
Fundamental tests performed.
DEPENDENCIES:
PR #973 and it's follow-up PR
ISSUE:
This solves issue #946
Issue to be solved: #991
CHECKLIST
LABELS (optional):
A Code Manager needs to add the following labels to this PR: