Skip to content

[develop] Update CI/CD pipeline to work properly on Gaea C6 - #1384

Merged
MichaelLueken merged 2 commits into
ufs-community:developfrom
MichaelLueken:bf/GaeaC6_CICD_fix
Sep 4, 2026
Merged

[develop] Update CI/CD pipeline to work properly on Gaea C6#1384
MichaelLueken merged 2 commits into
ufs-community:developfrom
MichaelLueken:bf/GaeaC6_CICD_fix

Conversation

@MichaelLueken

@MichaelLueken MichaelLueken commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

DESCRIPTION OF CHANGES:

  • .cicd/scripts/wrapper_srw_ftest.sh - Updated job_id to include | awk -F';' '{print $1}' to allow the Functional WorkflowTaskTests stage to run on Gaea C6 following the SLURM update

Type of change

  • Bug fix (non-breaking change which fixes an issue)

TESTS CONDUCTED:

  • gaeac6.intel
  • Jenkins
  • coverage test suite

DEPENDENCIES:

None

DOCUMENTATION:

None

ISSUE:

N/A

CHECKLIST

  • My code follows the style guidelines in the Contributor's Guide
  • I have performed a self-review of my own code using the Code Reviewer's Guide
  • My changes do not require updates to the documentation (explain).
    • This is an update to the Jenkins CI/CD scripts
  • My changes generate no new warnings
  • New and existing tests pass with my changes

CONTRIBUTORS

@kbooker79 - for discovering the appended ;c6 at the end of the job_id on Gaea C6 following the SLURM update and providing the fix.

* .cicd/scripts/wrapper_srw_ftest.sh - Updated job_id to include | awk -F';' '{print $1}' to allow the Functional WorkflowTaskTests stage to run on Gaea C6 following the SLURM update
@MichaelLueken

MichaelLueken commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator Author

Since the SLURM upgrade on Gaea C6, the Functional WorkflowTaskTests have been failing with messages similar to the following:

Running: sbatch -A epic --parsable /gpfs/f6/bil-fire8/scratch/Michael.Lueken/ufs-srweather-app/gaeac6/.cicd/scripts/sbatch_srw_ftest.sh
Waiting ten seconds for node to initialize
squeue: error: Invalid job id: 220189234;c6
Job has completed.
Missing results file! 
exit 1

During the SLURM upgrade, the Job ID was updated to include a semi-colon and the cluster the job was submitted on.

Setting the Job ID to just the Job ID and removing the semi-colon and the cluster, allows the Functional WorkflowTaskTests to run on the machine:

Running: sbatch -A epic --parsable /gpfs/f6/bil-fire8/scratch/Michael.Lueken/ufs-srweather-app/gaeac6/.cicd/scripts/sbatch_srw_ftest.sh
Waiting ten seconds for node to initialize
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes

@MichaelLueken

Copy link
Copy Markdown
Collaborator Author

The Gaea C6 Jenkins testing has successfully passed following the update in this PR.

Please review and approve this PR so that the nightly testing can once again resume on Gaea C6.

@MichaelLueken
MichaelLueken marked this pull request as draft August 18, 2026 17:48
* .cicd/scripts/wrapper_srw_ftest.sh - Add a separate run for Derecho,  since the fix doesn't work with PBS
@MichaelLueken

Copy link
Copy Markdown
Collaborator Author

The update to wrapper_srw_ftest.sh works on SLURM, but fails on Derecho (PBS). Updated to maintain current job submission on Derecho, and apply the fix for SLURM.

@MichaelLueken

Copy link
Copy Markdown
Collaborator Author

The update at hash 2ac20d3 allows the Functional WorkflowTaskTests to run on Derecho:

Running: qsub -A NRAL0032  /glade/derecho/scratch/mlueken/ufs-srweather-app/derecho/.cicd/scripts/qsub_srw_ftest.sh
Waiting ten seconds for node to initialize
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
qstat: 7145594.desched1 Job has finished, use -x or -H to obtain historical job information
Job has completed.
exit 0

It is also passing on Gaea C6:

Running: sbatch -A epic --parsable /gpfs/f6/bil-fire8/scratch/Michael.Lueken/ufs-srweather-app/gaeac6/.cicd/scripts/sbatch_srw_ftest.sh
Waiting ten seconds for node to initialize
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job has completed.
exit 0

Reopening PR now.

@MichaelLueken
MichaelLueken marked this pull request as ready for review August 18, 2026 18:58

@RatkoVasic-NOAA RatkoVasic-NOAA left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[Ratko.Vasic@gaea65:/gpfs/f6/bil-fire8/world-shared/Ratko.Vasic/ufs-srweather-app/gaeac6/.cicd/scripts]# ./wrapper_srw_ftest.sh
Running: sbatch -A epic --parsable /gpfs/f6/bil-fire8/world-shared/Ratko.Vasic/ufs-srweather-app/gaeac6/.cicd/scripts/sbatch_srw_ftest.sh
Waiting ten seconds for node to initialize
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job has completed.
exit 0

Approved.

@natalie-perlin

Copy link
Copy Markdown
Collaborator

@RatkoVasic-NOAA @MichaelLueken - how do I run this test on gaeac6?
This does not work running from a terminal on Gaeac6 for me. Do I need to run it from Jenkins site?

@MichaelLueken

Copy link
Copy Markdown
Collaborator Author

@RatkoVasic-NOAA @MichaelLueken - how do I run this test on gaeac6? This does not work running from a terminal on Gaeac6 for me. Do I need to run it from Jenkins site?

@natalie-perlin - please follows these steps to clone my branch and run the Functional WorkflowTaskTests:

  1. You will need to clone the branch so that it is ufs-srweather-app/gaeac6 (i.e., git clone -b bf/GaeaC6_CICD_fix git@github.com:MichaelLueken/ufs-srweather-app.git ufs-srweather-app/gaeac6)
  2. You will need to set the following environmental variables:
  • SRW_PLATFORM="gaeac6"
  • SRW_COMPILER="intel"
  • WORKSPACE= "path_to_ufs-srweather_app" (i.e., "/gpfs/f6/bil-fire8/scratch/Michael.Lueken/ufs-srweather-app")
  • SRW_PROJECT="epic"
  1. cd .cicd/scripts
  2. ./srw_build.sh (to build the SRW using the cicd scripts)
  3. ./wrapper_srw_ftest.sh (to run the Functional WorkflowTaskTests)

Please let me know if you have any questions or encounter issues.

@natalie-perlin

Copy link
Copy Markdown
Collaborator

All done!.. Worked well on gaeac6:

[Natalie.Perlin@gaea62:/gpfs/f6/bil-fire8/scratch/Natalie.Perlin/SRW/ufs-srweather-app/gaeac6/.cicd/scripts]$ ./wrapper_srw_ftest.sh 
Running: sbatch -A epic --parsable /gpfs/f6/bil-fire8/scratch/Natalie.Perlin/SRW/ufs-srweather-app/gaeac6/.cicd/scripts/sbatch_srw_ftest.sh
Waiting ten seconds for node to initialize
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job is still running. Check again in two minutes
Job has completed.
exit 0

Approving it now.

@natalie-perlin natalie-perlin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, approved!

@MichaelLueken
MichaelLueken merged commit 570c047 into ufs-community:develop Sep 4, 2026
7 of 8 checks passed
@github-project-automation github-project-automation Bot moved this from In Review to Done in SRW App's PR Reviewers Sep 4, 2026
@MichaelLueken
MichaelLueken deleted the bf/GaeaC6_CICD_fix branch September 4, 2026 13:35
natalie-perlin added a commit to natalie-perlin/ufs-srweather-app that referenced this pull request Sep 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority: HIGH run_we2e_coverage_tests Run the coverage set of SRW end-to-end tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants