Skip to content

apply ci path patch only when running g-w ctests#1554

Merged
RussTreadon-NOAA merged 1 commit into
developfrom
bugfix/ci
Mar 18, 2025
Merged

apply ci path patch only when running g-w ctests#1554
RussTreadon-NOAA merged 1 commit into
developfrom
bugfix/ci

Conversation

@RussTreadon-NOAA
Copy link
Copy Markdown
Contributor

@RussTreadon-NOAA RussTreadon-NOAA commented Mar 18, 2025

Description

This PR resolves an undefined variable bug in ci/run_ci.sh

Companion PRs

None

Issues

Resolves #1553

Automated CI tests to run in Global Workflow

  • atm_jjob
  • C96C48_ufs_hybatmDA
  • C96C48_hybatmaerosnowDA
  • C48mx500_3DVarAOWCDA
  • C48mx500_hybAOWCDA
  • C96C48_hybatmDA

@RussTreadon-NOAA RussTreadon-NOAA self-assigned this Mar 18, 2025
@RussTreadon-NOAA RussTreadon-NOAA added the hera-RT Queue for automated testing on Hera label Mar 18, 2025
@emcbot emcbot added hera-RT-Running Automated testing running on Hera and removed hera-RT Queue for automated testing on Hera labels Mar 18, 2025
@RussTreadon-NOAA RussTreadon-NOAA added the hercules-RT Queue for automated testing on Hercules label Mar 18, 2025
@emcbot
Copy link
Copy Markdown

emcbot commented Mar 18, 2025

Automated GDASApp Testing Results:
Machine: hera

Start: Tue Mar 18 13:20:04 UTC 2025 on hfe10
---------------------------------------------------
Build:                                 *SUCCESS*
Build: Completed at Tue Mar 18 14:06:49 UTC 2025
---------------------------------------------------
Tests: ctest -j12 -R gdasapp
Tests:                                 *SUCCESS*
Tests: Completed at Tue Mar 18 14:07:56 UTC 2025
Tests: 100% tests passed, 0 tests failed out of 32

@emcbot emcbot added hera-RT-Passed Automated testing successful on Hera and removed hera-RT-Running Automated testing running on Hera labels Mar 18, 2025
@RussTreadon-NOAA RussTreadon-NOAA added the orion-RT Queue for automated testing on Orion label Mar 18, 2025
@emcbot emcbot added hercules-RT-Running Automated testing running on Hercules orion-RT-Running Automated testing running on Orion and removed hercules-RT Queue for automated testing on Hercules orion-RT Queue for automated testing on Orion labels Mar 18, 2025
@emcbot
Copy link
Copy Markdown

emcbot commented Mar 18, 2025

Automated GDASApp Testing Results:
Machine: hercules

Start: Tue Mar 18 10:46:29 CDT 2025 on hercules-login-1.hpc.msstate.edu
---------------------------------------------------
Build:                                 *SUCCESS*
Build: Completed at Tue Mar 18 11:19:03 CDT 2025
---------------------------------------------------
Tests: ctest -j12 -R gdasapp
Tests:                                 *SUCCESS*
Tests: Completed at Tue Mar 18 11:20:08 CDT 2025
Tests: 100% tests passed, 0 tests failed out of 32

@emcbot emcbot added hercules-RT-Passed Automated testing successful on Hercules and removed hercules-RT-Running Automated testing running on Hercules labels Mar 18, 2025
@RussTreadon-NOAA RussTreadon-NOAA changed the title apply ci path patch only when running g-w ctests (#1553) apply ci path patch only when running g-w ctests Mar 18, 2025
@emcbot
Copy link
Copy Markdown

emcbot commented Mar 18, 2025

Automated GDASApp Testing Results:
Machine: orion

Start: Tue Mar 18 10:46:50 AM CDT 2025 on orion-login-1.hpc.msstate.edu
---------------------------------------------------
Build:                                 *SUCCESS*
Build: Completed at Tue Mar 18 11:47:00 AM CDT 2025
---------------------------------------------------
Tests: ctest -j12 -R gdasapp
Tests:                                 *SUCCESS*
Tests: Completed at Tue Mar 18 11:58:31 AM CDT 2025
Tests: 100% tests passed, 0 tests failed out of 32

@emcbot emcbot added orion-RT-Passed Automated testing successful on Orion and removed orion-RT-Running Automated testing running on Orion labels Mar 18, 2025
@RussTreadon-NOAA
Copy link
Copy Markdown
Contributor Author

RussTreadon-NOAA commented Mar 18, 2025

The modified ci/run_ci.sh was manually installed in the role.jedipara (Hera) and role-da (MSU) accounts. GDASApp RT was triggered via labels. GDASApp tests passed on Hera, Hercules, and Orion.

This PR is ready for review.

CoryMartin-NOAA
CoryMartin-NOAA reviewed Mar 18, 2025
View reviewed changes
Comment thread ci/run_ci.sh
@RussTreadon-NOAA
Copy link
Copy Markdown
Contributor Author

RussTreadon-NOAA commented Mar 18, 2025

Thank you, @CoryMartin-NOAA and @DavidNew-NOAA . Merging now.

@RussTreadon-NOAA RussTreadon-NOAA merged commit c84c4e5 into develop Mar 18, 2025
@RussTreadon-NOAA RussTreadon-NOAA deleted the bugfix/ci branch March 18, 2025 17:35
jspsl pushed a commit that referenced this pull request Mar 26, 2025
@RussTreadon-NOAA @jspsl
apply ci path patch only when running g-w ctests (#1554)
6689a04
danholdaway added a commit that referenced this pull request Mar 26, 2025
@danholdaway
Merge branch 'develop' into feature/remove_var_name_aliasing
10db435 
* develop:
  added original date (#1570)
  Update OCEAN_BASIN_FILE_ORION path for marine ctest (#1569)
  Fixing window_begin and end in yaml for redating ADT (#1565)
  Save LETKF mean analysis and mean increment for verification (#1566)
  Update JEDI hashes (20250324) (#1567)
  Removes obsolete json files from marine bufr2ioda (#1564)
  Add hybrid 4denvar as a phase3 validation configuration (#1552)
  Bugfix/ocn ecen (#1550)
  Bugfix SSS obs for LETKF (#1560)
  Bugfix ocean ensemble recentering (#1558)
  apply ci path patch only when running g-w ctests (#1554)
  Update JEDI hashes (20250317)  (#1531)
  add option to create_experiment.py in create_exp.sh  (#1546)
  Update jcb hashes and support initial IODA stats capability (#1547)
  Fix GW-CI CTest dependencies (#1534)
  Converts prepoceanobs to use JCB-style obs list and config files (#1527)
  JEDI-based ensemble recentering and analysis calculation (#1488)
  tools for phase3 validation (#1526)
  Update JEDI hashes (20250306) (#1520)
  New jcb-gdas # and insitu obs prep bug fix  (#1523)
  Marine in situ obs update (SSS, moorings, drifters) (#1511)
  Save marine background and analysis spread (#1517)
  Minor build and install updates and ctest cleanup (#1512)
  correct logic for upper loop limit in chem_diagb (#1516)
  Incorporate multi-cycle concatination and separation of variables in post-BUFR observations (#1504)
  Update JEDI hashes (20250225) (#1475)
  Add a depth dependent e-fold scale (#1494)
  Update jcb and da-utils hashes to support land DA. (#1509)
  Add options to inflate the ensemble while recentering (#1501)
  Add components to use jcb yamls for marine letkf job (#1486)
  Add GNSSRO bufr2ioda and json files (#1448)
  Removal of the marine verification tools (#1499)
  update logic to identify MSU machine (#1498)
  IODA Converter with In-Situ Observations Concatenation and ObsError Inflation (#1472)
  Add sea ice recentering (#1478)
  marine vrfy bugfix (#1483)
  add LD_LIBRARY_PATH patch for wcoss2 build (#1484)
  Bugfix for ensemble recenter + add a test (#1477)
  Add code changes for the implementation of the  low-resolution B (#1441)
  update spack-stack path in Hera modulefiles (#1471)
  fixed glider output filename (#1469)
  add gdas_waveinit to C48mx500_3DVarAOWCDA ctest suite (#1470)
  update jcb-gdas hash 20250128 (#1466)
  Time series of omb stats and html doc (#1463)
  Update marine vrfy HTML codes (#1464)
  Automate the verify tools to submit from start pdy to end pdy (#1458)
  Update an offline marine vrfy tool (#1447)
  Update JEDI hashes (20250127) (#1460)
DavidNew-NOAA pushed a commit that referenced this pull request Jan 16, 2026
@RussTreadon-NOAA
apply ci path patch only when running g-w ctests (#1554)
c55ff6b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@CoryMartin-NOAA CoryMartin-NOAA CoryMartin-NOAA approved these changes

@DavidNew-NOAA DavidNew-NOAA DavidNew-NOAA approved these changes

Assignees

@RussTreadon-NOAA RussTreadon-NOAA

Labels

hera-RT-Passed Automated testing successful on Hera hercules-RT-Passed Automated testing successful on Hercules orion-RT-Passed Automated testing successful on Orion

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Restrict CI g-w path patch to g-w tests

4 participants

@RussTreadon-NOAA @emcbot @CoryMartin-NOAA @DavidNew-NOAA