NOAA-EMC / global-workflow

Global Superstructure/Workflow supporting the Global Forecast System (GFS)

https://global-workflow.readthedocs.io/en/latest

GNU Lesser General Public License v3.0

70 stars 162 forks source link

Gaea c5 added to Jenkinsfile #2664

Open AnilKumar-NOAA opened 3 weeks ago

AnilKumar-NOAA commented 3 weeks ago

Description

Added Gaea c5 to Jenkinsfile

Type of change

Bug fix (fixes something broken)
New feature (adds functionality)
Maintenance (code refactor, clean-up, new CI test, etc.)

Change characteristics

Is this a breaking change (a change in existing functionality)? YES/NO
Does this change require a documentation update? YES/NO

How has this been tested?

Checklist

[ ] Any dependent changes have been merged and published
[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] My changes generate no new warnings
[ ] New and existing tests pass with my changes
[ ] I have made corresponding changes to the documentation if necessary

TerrenceMcGuinness-NOAA commented 3 weeks ago

@AnilKumar-NOAA Add - gaea on skip_ci_on_hosts: for every case yaml file in the $HOMEgfs/ci/cases/pr directory that is not being test on C5 Gaea

TerrenceMcGuinness-NOAA commented 3 weeks ago

@AnilKumar-NOAA Add a config.gaea file in $HOMEgfs/ci/platforms like the other ones with the allocations account HPC_ACCOUNT that will be running the cases you have designated above.

JENKINS_AGENT_LANUCH_DIR and JENKINS_WORK_DIR will also need to be designated and will become clear when you receive the instructions from Kris Booker about starting the Jenkins Java Agent on Gaea.

AnilKumar-NOAA commented 3 weeks ago

Left instructions in the Conversations tab for a few more files that need to be updated and added. @DavidBurrows-NCO @AnilKumar-NOAA David has made these changes described above in PR #2577. You guys need to combined these into one PR. I will meetup with you to for a short technical brief as we workout which role account these are going to run on.

This PR has everything for ci run. we may discard other PR and focus on this one. Thanks @WalterKolczynski-NOAA @DavidBurrows-NCO

WalterKolczynski-NOAA commented 3 weeks ago

I made the Gaea labels this morning. Let me know if you need anything else.

TerrenceMcGuinness-NOAA commented 2 weeks ago

PR updates look good. I will add a PR to this branch too for the needed updates so the Jenkinsfile the EPIC team can use the Gaea Node in Jenkins without the EMC tag. What remains then would be the token for GitHub CLI access to the emcbot account in the epic role account that is running the Java Agent for the Gaea Node.

AnilKumar-NOAA commented 2 weeks ago

@TerrenceMcGuinness-NOAA Thanks Terry! Will check with kris and get token access for github CLI. Thanks