Containerize unit testing CI jobs in MPAS dynamical core#489
Open
kuanchihwang wants to merge 5 commits intoESCOMP:developmentfrom
Open
Containerize unit testing CI jobs in MPAS dynamical core#489kuanchihwang wants to merge 5 commits intoESCOMP:developmentfrom
kuanchihwang wants to merge 5 commits intoESCOMP:developmentfrom
Conversation
kuanchihwang
temporarily deployed
to
CI-tests-on-CIRRUS
GitHub Actions
Inactive
kuanchihwang
force-pushed
the
staging/mpas-dycore-container-ci
branch
from
aef5960 to
6b2182e
Compare
kuanchihwang
temporarily deployed
to
CI-tests-on-CIRRUS
GitHub Actions
Inactive
* When `-warn all` is specified, the unit tests fail to compile with errors like: error #7977: The type of the function reference does not match the type of the function definition. The errors are found to occur in pFUnit-generated code, which is outside of our control. Work around it with `-warn all,nointerfaces`. * Placing `use funit` in the smallest scoping unit causes excessive compile time for Intel compilers. Compiling the unit tests can take 20-30 minutes, which is absolutely not normal. Work around it by moving `use funit` to the module level as an exception to the coding standards.
Some CI jobs are very light on computing resources. Migrate those to the newly available `ubuntu-slim` runner (1 vCPU). As a result, queue time should be improved.
Due to the deprecation of Node.js 20 actions, update GitHub actions to the latest versions that use Node.js 24. The deadline is September 16th, 2026.
kuanchihwang
force-pushed
the
staging/mpas-dycore-container-ci
branch
from
6b2182e to
54ef1f5
Compare
kuanchihwang
temporarily deployed
to
CI-tests-on-CIRRUS
GitHub Actions
Inactive
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Tag name (required for release branches)
None
Originator(s)
kuanchihwang
Description (include the issue title, and the keyword ['closes', 'fixes', 'resolves'] followed by the issue number)
This PR containerizes the unit testing CI jobs in MPAS dynamical core with an expanded compiler matrix (GNU 11-15, Intel oneAPI 2024, and 2025). The pFUnit bootstrap overhead has been eliminated.
In addition, this PR migrates lightweight CI jobs to the
ubuntu-slimrunner and updates GitHub actions to latest versions for Node.js 24 compatibility.Describe any changes made to the build system
None
Describe any changes made to the namelist
None
List any changes to the defaults for the input datasets (e.g., boundary datasets)
None
List all files eliminated and why
None
List all files added and what they do
None
List all existing files that have been modified, and describe the changes
Regression tests
Known failing test.
Known failing test.
Pass.