-
Notifications
You must be signed in to change notification settings - Fork 168
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
Add Gaea C5 to CI #2814
Add Gaea C5 to CI #2814
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.
Looks good!
@WalterKolczynski-NOAA - Please check this PR for merge. Terry is the one reviewer and need your approval for final merge. Thanks |
Title and description both say Jenkinsfile, but I see no Jenkinsfile changes. |
Initially, we made changes to the Jenkinsfile, adding Gaea in several places. However, Terry later updated the Jenkinsfile himself, incorporating the Gaea additions (nodename and custom_workspace) and pushed those changes to my repo, which is why they're not reflected in this PR. |
@AnilKumar-NOAA I've updated them this time. Next time, please update them yourself to be consistent with what the PR is actually doing. |
…e_rocoto * origin/develop: Jenkins Pipeline Updates (NOAA-EMC#2815) Add Gaea C5 to CI (NOAA-EMC#2814) Add support for forecast-only runs on AWS (NOAA-EMC#2711) Add fixes to products for when REPLAY IC's are used (NOAA-EMC#2755) Add capability to run forecast in segments (NOAA-EMC#2795)
CI Testing and Jenkinsfile for Gaea C5
Description
Adds gaea as a recognized machine in the CI system.
Type of change
Change characteristics
How has this been tested?
Checklist