CI JJOB Tests using CMake #3214
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
update last bit
fix misspelled work with create
fixed a grammar in README.md
path to source as HOMEgfs
|
Execute test is failing when I try |
aerorahul
reviewed
Jan 14, 2025
ctests/CMakeLists.txt
Outdated
| # Set HPC_ACCOUNT | ||
| set_from_env_or_default(HPC_ACCOUNT HPC_ACCOUNT " ") | ||
| if (NOT DEFINED HPC_ACCOUNT) | ||
| message(FATAL_ERROR "HPC_ACCOUNT must be set") |
There was a problem hiding this comment.
FATAL_ERROR will stop cmake processing (I think).
The return() after this line will not allow the cmake-ification of the project.
aerorahul
reviewed
Jan 14, 2025
aerorahul
reviewed
Jan 14, 2025
aerorahul
reviewed
Jan 14, 2025
aerorahul
reviewed
Jan 14, 2025
Co-authored-by: Rahul Mahajan <[email protected]>
… needed env vars set
|
I'm still getting the same error when I try to run on Hercules:
|
|
@WalterKolczynski-NOAA
|
Okay, I ran on the same machine so I was expecting it to stage from Terry's space. |
WalterKolczynski-NOAA
approved these changes
Jan 16, 2025
tsga
added a commit
to tsga/global-workflow
that referenced
this pull request
Jan 22, 2025
* develop: Add echgres as a dependency only for RUN=enkfgdas, not enkfgfs (NOAA-EMC#3246) Add domain level to wave gridded COM path (NOAA-EMC#3137) CI JJOB Tests using CMake (NOAA-EMC#3214) Make assorted updates to waves (NOAA-EMC#3190) Move WCOSS2 LD_LIBRARY_PATH patches to load_ufsda_modules.sh (NOAA-EMC#3236) Adding a gefs_arch task to GEFS workflow (NOAA-EMC#3211) Add additional GEFS variables needed for AI/ML applications (NOAA-EMC#3221) Add bmat task dependency to marine LETKF task (NOAA-EMC#3224) Resolve bug with LMOD_TMOD_FIND_FIRST setting affecting build on WCOSS2 (NOAA-EMC#3229) Reinstate product groups (NOAA-EMC#3208) Additional fixes for downstream jobs (NOAA-EMC#3187) Turn IAU off during staging job for cold start experiments (NOAA-EMC#3215) Update the gdas.cd hash and enable GDASApp to run on WCOSS2 (NOAA-EMC#3220) Update upload-artifact to v4 (NOAA-EMC#3216) Prevent duplicate case generation in generate_workflows.sh (NOAA-EMC#3217) Update g-w to cycle with C1152 ATM (NOAA-EMC#3206) Separate use of initial increment/perturbation file from REPLAY/+03 ICs (NOAA-EMC#3119) Update gsi_enkf hash and gsi_ver (NOAA-EMC#3207) Remove cpus-per-task from APRUN_OCNANALECEN on WCOSS2 (NOAA-EMC#3212) Remove 5WAVH from AWIPS GRIB2 parm files (NOAA-EMC#3146) Remove multi-grid wave support (NOAA-EMC#3188) Add echgres as a dependency for earc (NOAA-EMC#3202)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Adding CI tests at the JJOB level using CMake/ctest wrappers:
These individual JJOB tests have four distinct phases:
${HOMEgfs}/ci/ctest/cases/{CASE}_{JJOB}.yamlResolves #3204
Type of change
Change characteristics
How has this been tested?
Ran with CMake/ctest:
Checklist