Skip to content
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

Update JEDI hashes to include Model Variable Renaming Sprint #1355

Merged
merged 19 commits into from
Nov 15, 2024

Conversation

RussTreadon-NOAA
Copy link
Contributor

@RussTreadon-NOAA RussTreadon-NOAA commented Oct 30, 2024

This PR updates JEDI hashes in sorc/ to bring in changes associated with the Model Variable Renaming Sprint. This is required to fully resolve #1313.

Resolves #1313
Resolves #1362

Depends: jcb-gdas #45

@RussTreadon-NOAA RussTreadon-NOAA self-assigned this Oct 30, 2024
@RussTreadon-NOAA RussTreadon-NOAA added the hera-GW-RT Queue for automated testing with global-workflow on Hera label Oct 30, 2024
@emcbot emcbot added hera-GW-RT-Running Automated testing with global-workflow running on Hera and removed hera-GW-RT Queue for automated testing with global-workflow on Hera labels Oct 30, 2024
@emcbot
Copy link

emcbot commented Oct 30, 2024

Automated Global-Workflow GDASApp Testing Results:
Machine: hera

Start: Wed Oct 30 16:11:17 UTC 2024 on hfe10
---------------------------------------------------
Build:                                 *SUCCESS*
Build: Completed at Wed Oct 30 16:11:48 UTC 2024
---------------------------------------------------
Tests:                                 *SUCCESS*
Tests: Completed at Wed Oct 30 16:11:51 UTC 2024
Tests: 

@emcbot emcbot added hera-GW-RT-Passed Automated testing with global-workflow successful on Hera and removed hera-GW-RT-Running Automated testing with global-workflow running on Hera labels Oct 30, 2024
@RussTreadon-NOAA RussTreadon-NOAA added hera-RT Queue for automated testing on Hera and removed hera-GW-RT-Passed Automated testing with global-workflow successful on Hera labels Oct 30, 2024
@emcbot
Copy link

emcbot commented Oct 30, 2024

Automated GDASApp Testing Results:
Machine: hera

Start: Wed Oct 30 18:10:20 UTC 2024 on hfe04
---------------------------------------------------
Build:                                  *FAILED*
Build: Failed at Wed Oct 30 19:08:03 UTC 2024
Build: see output at /scratch1/NCEPDEV/da/role.jedipara/CI/GDASApp/PR/1355/GDASApp/log.build

@emcbot emcbot added the hera-RT-Passed Automated testing successful on Hera label Oct 30, 2024
@RussTreadon-NOAA RussTreadon-NOAA added hera-RT Queue for automated testing on Hera and removed hera-RT Queue for automated testing on Hera hera-RT-Passed Automated testing successful on Hera labels Oct 30, 2024
@emcbot emcbot added hera-RT-Running Automated testing running on Hera and removed hera-RT Queue for automated testing on Hera labels Oct 30, 2024
@RussTreadon-NOAA RussTreadon-NOAA added hera-RT Queue for automated testing on Hera and removed hera-RT-Running Automated testing running on Hera labels Oct 30, 2024
@emcbot emcbot added hera-RT-Running Automated testing running on Hera and removed hera-RT Queue for automated testing on Hera labels Oct 30, 2024
@emcbot
Copy link

emcbot commented Oct 30, 2024

Automated GDASApp Testing Results:
Machine: hera

Start: Wed Oct 30 19:40:23 UTC 2024 on hfe04
---------------------------------------------------
Build:                                 *SUCCESS*
Build: Completed at Wed Oct 30 20:32:02 UTC 2024
---------------------------------------------------

@emcbot emcbot added hera-RT-Failed Automated testing failed on Hera and removed hera-RT-Running Automated testing running on Hera labels Oct 30, 2024
@RussTreadon-NOAA RussTreadon-NOAA added hera-RT Queue for automated testing on Hera and removed hera-RT-Failed Automated testing failed on Hera labels Oct 31, 2024
@emcbot emcbot added hera-RT-Running Automated testing running on Hera and removed hera-RT Queue for automated testing on Hera labels Oct 31, 2024
@emcbot
Copy link

emcbot commented Oct 31, 2024

Automated GDASApp Testing Results:
Machine: hera

Start: Thu Oct 31 10:24:14 UTC 2024 on hfe03
---------------------------------------------------
Build:                                 *SUCCESS*
Build: Completed at Thu Oct 31 11:12:02 UTC 2024
---------------------------------------------------
Tests:                                  *Failed*
Tests: Failed at Thu Oct 31 11:14:12 UTC 2024
Tests: 76% tests passed, 8 tests failed out of 33
	1953 - test_gdasapp_bufr2ioda_insitu_profile_argo (Failed)
	1954 - test_gdasapp_bufr2ioda_insitu_profile_bathy (Failed)
	1955 - test_gdasapp_bufr2ioda_insitu_profile_glider (Failed)
	1956 - test_gdasapp_bufr2ioda_insitu_profile_tesac (Failed)
	1957 - test_gdasapp_bufr2ioda_insitu_profile_tropical (Failed)
	1958 - test_gdasapp_bufr2ioda_insitu_profile_xbtctd (Failed)
	1959 - test_gdasapp_bufr2ioda_insitu_surface_drifter (Failed)
	1960 - test_gdasapp_bufr2ioda_insitu_surface_trkob (Failed)
Tests: see output at /scratch1/NCEPDEV/da/role.jedipara/CI/GDASApp/PR/1355/GDASApp/build/log.ctest

@RussTreadon-NOAA
Copy link
Contributor Author

Will keep this PR in draft mode until GDASApp and jcb-gdas yamls and templates work with updated JEDI hashes.

@RussTreadon-NOAA RussTreadon-NOAA marked this pull request as ready for review November 15, 2024 19:33
@RussTreadon-NOAA
Copy link
Contributor Author

The changes in this PR have been tested in g-w PR #2992.

Tests completed include:

  1. test_gdasapp
    • 64/64 tests pass on Hera, Hercules, and Orion
  2. g-w DA CI
    • four configurations tested: GSI, JEDI ATM, JEDI Aerosol/Snow, JEDI WCDA
    • all four configurations successfully run all jobs on Hera, Hercules, and Orion
    • GSI and JED ATM CI work _as is" on Dogwood (WCOSS2)
    • JEDI Aerosol/Snow DA and JEDI WCDA work on Dogwood when GDASApp branch feature/resume_nightly' (this PR) is built with spack-stack/1.6.0`

@RussTreadon-NOAA
Copy link
Contributor Author

This PR SHOULD NOT be merged into develop until jcb-gdas PR #45 is approved and merged into jcb-gdas develop. Once this is done, the parm/jcb-gdas hash in feature/resume_nightly should be updated to the new jcb-gdas develop hash.

@RussTreadon-NOAA
Copy link
Contributor Author

@guillaumevernieres , @AndrewEichmann-NOAA , @DavidNew-NOAA, @CoryMartin-NOAA - this PR is ready for final review. It's been tested in g-w #2992.

Copy link
Collaborator

@DavidNew-NOAA DavidNew-NOAA left a comment

Choose a reason for hiding this comment

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

Looks good, especially given all the testing. Nice work, @RussTreadon-NOAA

@RussTreadon-NOAA
Copy link
Contributor Author

Thank you @DavidNew-NOAA and @CoryMartin-NOAA for the quick approvals. I'll go ahead and merge.

@RussTreadon-NOAA RussTreadon-NOAA merged commit e514b92 into develop Nov 15, 2024
5 checks passed
@RussTreadon-NOAA RussTreadon-NOAA deleted the feature/resume_nightly branch November 15, 2024 19:56
@WalterKolczynski-NOAA WalterKolczynski-NOAA added hera-GW-RT-Failed Automated testing with global-workflow failed on Hera and removed hera-GW-RT-Failed Automated testing with global-workflow failed on Hera labels Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hera-GW-RT-Failed Automated testing with global-workflow failed on Hera hera-RT-Failed Automated testing failed on Hera
Projects
None yet
5 participants