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

Enable GDASApp g-w CI cases to run on wcoss2 #3399

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

RussTreadon-NOAA
Copy link
Contributor

@RussTreadon-NOAA RussTreadon-NOAA commented Feb 26, 2025

Description

This PR removes wcoss2 from the skip_ci_on_hosts section of following g-w CI case yamls

  • C48mx500_3DVarAOWCDA
  • C48mx500_hybAOWCDA
  • C96C48_hybatmaerosnowDA
  • C96C48_ufs_hybatmDA

Each of these cases exercise various GDASApp capabilities.

Resolves #3397

Type of change

  • New feature - enables GDASApp g-w CI cases on wcoss2

Change characteristics

  • Is this a breaking change (a change in existing functionality)? NO
  • Does this change require a documentation update? NO
  • Does this change require an update to any of the following submodules? NO

How has this been tested?

  • Clone, build, and run g-w CI on WCOSS2 (Cactus)

Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • New and existing tests pass with my changes
  • This change is covered by an existing CI test or a new one has been added

@RussTreadon-NOAA
Copy link
Contributor Author

Place this PR in draft mode until g-w CI is run on Cactus

@RussTreadon-NOAA RussTreadon-NOAA marked this pull request as draft February 26, 2025 21:40
@RussTreadon-NOAA
Copy link
Contributor Author

WCOSS2 g-w CI

Install RussTreadon-NOAA:feature/wcoss2_gdasapp_ci at 5e11f42 on Cactus.

As noted in PR #3295 parm/config/gfs/config.aeroanlgenb was updated in the local copy of feature/wcoss2_gdasapp_ci so that the C96C48_hybatmaerosnowDA 00Z gdas_snowanl and enkfgdas_esnowanl jobs complete. PR #3295 contains this change but this PR has not yet been merged into g-w develop.

Run g-w CI with the following results

/lfs/h2/emc/ptmp/russ.treadon/EXPDIR/C48_ATM_wcoss2_gdasapp_ci
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202103231200        Done    Feb 27 2025 01:14:00    Feb 27 2025 02:25:26
 
/lfs/h2/emc/ptmp/russ.treadon/EXPDIR/C48mx500_3DVarAOWCDA_wcoss2_gdasapp_ci
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202103241800        Done    Feb 27 2025 01:14:02    Feb 27 2025 01:35:39
202103250000        Done    Feb 27 2025 01:14:02    Feb 27 2025 03:25:27
 
/lfs/h2/emc/ptmp/russ.treadon/EXPDIR/C48mx500_hybAOWCDA_wcoss2_gdasapp_ci
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202103241800        Done    Feb 27 2025 01:14:04    Feb 27 2025 01:35:43
202103250000        Done    Feb 27 2025 01:14:04    Feb 27 2025 02:45:45
 
/lfs/h2/emc/ptmp/russ.treadon/EXPDIR/C48_S2SWA_gefs_wcoss2_gdasapp_ci
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202103231200        Done    Feb 27 2025 01:14:06    Feb 27 2025 03:25:34
 
/lfs/h2/emc/ptmp/russ.treadon/EXPDIR/C48_S2SW_wcoss2_gdasapp_ci
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202103231200        Done    Feb 27 2025 01:14:09    Feb 27 2025 02:35:51
 
/lfs/h2/emc/ptmp/russ.treadon/EXPDIR/C96_atm3DVar_wcoss2_gdasapp_ci
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202112201800        Done    Feb 27 2025 01:14:12    Feb 27 2025 01:35:58
202112210000        Done    Feb 27 2025 01:14:12    Feb 27 2025 03:55:35
202112210600        Done    Feb 27 2025 01:14:12    Feb 27 2025 03:35:34
 
/lfs/h2/emc/ptmp/russ.treadon/EXPDIR/C96C48_hybatmaerosnowDA_wcoss2_gdasapp_ci
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202112201200        Done    Feb 27 2025 01:14:15    Feb 27 2025 01:36:02
202112201800        Done    Feb 27 2025 01:14:15    Feb 27 2025 03:45:40
202112210000        Done    Feb 27 2025 01:14:15    Feb 27 2025 03:40:50
 
/lfs/h2/emc/ptmp/russ.treadon/EXPDIR/C96C48_hybatmDA_wcoss2_gdasapp_ci
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202112201800        Done    Feb 27 2025 01:14:17    Feb 27 2025 01:36:07
202112210000        Done    Feb 27 2025 01:14:17    Feb 27 2025 03:25:49
202112210600        Done    Feb 27 2025 01:14:17    Feb 27 2025 03:35:45
 
/lfs/h2/emc/ptmp/russ.treadon/EXPDIR/C96C48_ufs_hybatmDA_wcoss2_gdasapp_ci
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
202402231800        Done    Feb 27 2025 01:14:19    Feb 27 2025 01:36:13
202402240000        Done    Feb 27 2025 01:14:19    Feb 27 2025 04:25:29
202402240600        Done    Feb 27 2025 01:14:19    Feb 27 2025 04:40:43
 
/lfs/h2/emc/ptmp/russ.treadon/EXPDIR/C96mx100_S2S_wcoss2_gdasapp_ci
   CYCLE         STATE           ACTIVATED              DEACTIVATED     
199405010000        Done    Feb 27 2025 01:14:22    Feb 27 2025 02:01:08

All g-w CI cases successfully ran to completion. Of particular interest, all g-w CI cases that exercise GDASApp capabilities ran to completion.

@RussTreadon-NOAA
Copy link
Contributor Author

This PR is ready for review.

Copy link
Contributor

@aerorahul aerorahul left a comment

Choose a reason for hiding this comment

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

works for me

Copy link
Contributor

@CatherineThomas-NOAA CatherineThomas-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. Glad to see all the tests are passing with the aerogenb change. That PR has now been merged, so this should be good to go. Approve.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enable g-w CI cases that exercise GDASApp capabilities on WCOSS
3 participants