[dev/gfs.v17] Add informative error messages to all err_exit calls#4895
Conversation
…s and echo error messages
…to err_exit calls
…ls across multiple job scripts
…ge in JGDAS_AERO_ANALYSIS_GENERATE_BMATRIX
|
CI tests passed on wcoss2. |
JessicaMeixner-NOAA
left a comment
There was a problem hiding this comment.
I've looked at the wave updates and am good to go as long as CI passes on both gaea and wcoss2.
|
For the following jobs and script: I recommend including the elaborated error messages that were accepted in #4793, specifically those from commit 1e7692b. |
@ChristopherHill-NOAA Thanks for pointing that out. |
ChristopherHill-NOAA
left a comment
There was a problem hiding this comment.
Changes proposed for the atmospheric downstream product jobs are reflective of those in #4793, and are approved.
DavidNew-NOAA
left a comment
There was a problem hiding this comment.
Some suggestions for JEDI jobs
…ysis scripts for clarity and consistency
…ew WDQMS Python script for GSI diagnostic file processing
|
@DavidNew-NOAA Made the suggested changes. Thanks! |
CatherineThomas-NOAA
left a comment
There was a problem hiding this comment.
Thanks for incorporating my suggestions @AntonMFernando-NOAA. Approve.
Co-authored-by: David Huber <69919478+DavidHuber-NOAA@users.noreply.github.com>
Co-authored-by: David Huber <69919478+DavidHuber-NOAA@users.noreply.github.com>
Co-authored-by: David Huber <69919478+DavidHuber-NOAA@users.noreply.github.com>
|
Running tests on WCOSS2. |
|
All tests passed on WCOSS2. |
Description
This adds informative error messages to all err_exit calls. This was flagged as an issue for GCAFS and is being addressed across the board. It also removes err_exit calls from ush/ scripts in favor of exit calls.
Resolves (dev/gfs.v17) Add reason for failure into 'FATAL ERROR' message #4812
Resolves Add reason for failure into 'FATAL ERROR' message #4722
Type of change
Change characteristics
How has this been tested?