-
Notifications
You must be signed in to change notification settings - Fork 176
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
Removes no such file or directory
messages
#2146
Removes no such file or directory
messages
#2146
Conversation
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.
Update to address No such file or directory
in scripts/exglobal_diag.sh
is taken care of, as well as some additional shellcheck warning fixes (good). Surprised this script hasn't been touched earlier and had shellcheck stuff taken care of...oh well, fun for you!
Left a few comments, one of which involves even more shellcheck warning fixes. Thanks for working on this @HenryWinterbottom-NOAA !
@HenryWinterbottom-NOAA Have all of the scripts referenced in the main bugzilla description of #1252 been examined for "No such" statements and resolved if they still have them? If not (totally fine btw!), then please remove the "Resolves" statement so this PR doesn't close the associated issue just yet. Yourself or another developer will need to check the other jobs and confirm the issue is resolved before closing it...which will be a bit tricky for some jobs like the MOS jobs since we can't run them in our rocoto mesh (even after PR #2148 goes in). Thanks! |
Co-authored-by: Walter Kolczynski - NOAA <Walter.Kolczynski@noaa.gov>
Co-authored-by: Walter Kolczynski - NOAA <Walter.Kolczynski@noaa.gov>
Co-authored-by: Walter Kolczynski - NOAA <Walter.Kolczynski@noaa.gov>
Co-authored-by: Walter Kolczynski - NOAA <Walter.Kolczynski@noaa.gov>
Co-authored-by: Walter Kolczynski - NOAA <Walter.Kolczynski@noaa.gov>
Co-authored-by: Walter Kolczynski - NOAA <Walter.Kolczynski@noaa.gov>
Co-authored-by: Walter Kolczynski - NOAA <Walter.Kolczynski@noaa.gov>
Co-authored-by: Walter Kolczynski - NOAA <Walter.Kolczynski@noaa.gov>
Co-authored-by: Walter Kolczynski - NOAA <Walter.Kolczynski@noaa.gov>
|
|
Automated global-workflow Testing Results:
|
Automated global-workflow Testing Results:
|
|
|
Co-authored-by: Walter Kolczynski - NOAA <Walter.Kolczynski@noaa.gov>
@HenryWinterbottom-NOAA Please test this on your own on at least one machine before we run CI again. I don't want to do an iterative back-and-forth if there are more issues. |
@HenryWinterbottom-NOAA Have you successfully tested this so I can try the CI on it? |
@WalterKolczynski-NOAA I am working on it now. |
Will revisit this later |
Description
This PR addresses issue #1252. The following is accomplished:
ncdiag
file concatenation;shellcheck
test are resolved; where exceptions are made but no relevant, the respective check is disabled.Type of change
Change characteristics
How has this been tested?
A set of files, replicating the respective GSI
ncdiag
files, is created. The respective files are listed here. Files were selectively removed and the script sub-section was executed to verify that the erroneous messages, raising issue #1252 no longer raised the specified messages.Checklist