[Release-v2.1.0] Various updates for release#453
Conversation
MichaelLueken
left a comment
There was a problem hiding this comment.
This work is moving the UFS_SRW_App/develop contents into UFS_SRW_App/v2p1 for Gaea. This is the first step in completing issue #452. Approving this change.
|
@mark-a-potts Just wanting to check first, but are you hoping to modify the rest of the machine files with this PR, or is this ready to be committed? Thanks! |
|
No, it is not ready to be committed, yet. I am working through some of the errors that showed up in the comprehensive tests and trying to fix them where I can (mostly due to missing data or bad paths in machine files). I would also like to update all the "UFS_SRW_App/develop" instances to "UFS_SRW_App/v2p1" with this PR, but those changes won't work until the directories get created on all the machines. I'll be adding more commits to this over the weekend (maybe) and next Monday. |
|
You may want to add a |
Thanks @danielabdi-noaa, good catch! Just added it. |
|
I have finished the comprehensive tests on Cheyenne (gnu) with the only failures being due to exceeding wall-clock times. The tests are now also passing on AWS and @EdwardSnyder-NOAA indicates they passed on Azure and GCP, so I am going to go ahead and merge this. I think it will be the final changes to code other than some possible tweaks for generic linux/mac installs from @natalie-perlin. |
DESCRIPTION OF CHANGES:
Type of change
TESTS CONDUCTED:
CHECKLIST
LABELS (optional):
A Code Manager needs to add the following labels to this PR:
CONTRIBUTORS (optional):