-
Notifications
You must be signed in to change notification settings - Fork 108
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
Check for git clone failure in rt.sh #680
Labels
bug
Something isn't working
Comments
GeorgeGayno-NOAA
added a commit
to GeorgeGayno-NOAA/UFS_UTILS
that referenced
this issue
Aug 9, 2022
GeorgeGayno-NOAA
added a commit
to GeorgeGayno-NOAA/UFS_UTILS
that referenced
this issue
Aug 9, 2022
GeorgeGayno-NOAA
added a commit
to GeorgeGayno-NOAA/UFS_UTILS
that referenced
this issue
Aug 9, 2022
GeorgeGayno-NOAA
added a commit
to GeorgeGayno-NOAA/UFS_UTILS
that referenced
this issue
Aug 9, 2022
How to test this logic? The disk has been cleaned up on Hera. So I can't repeat the original problem. All I can do is adjust the script logic to trip the error. Example, look for the wrong directory:
Or make the clone fail:
|
The error was tripped on Hera, Jet, Orion and WCOSS2. As expected, the script execution stopped and a proper email was sent to my NOAA email account. |
Merged
GeorgeGayno-NOAA
added a commit
to GeorgeGayno-NOAA/UFS_UTILS
that referenced
this issue
Aug 9, 2022
GeorgeGayno-NOAA
added a commit
that referenced
this issue
Aug 9, 2022
Add logic to check if the git clone failed. Upon failure, exit script and send a clear email message. Fixes #680.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The
rt.sh
script is run off the cron on all supported machines. Recently, the script failed during thegit clone
command on Hera because the machine's disk was full. The log file showed:An email was sent to the maintainers, but the subject line did not indicate the machine, and the body of the email was blank.
The script needs better error handling.
The text was updated successfully, but these errors were encountered: