Skip to content

Updates to CI testing + support for Intel oneApi.#658

Open
dustinswales wants to merge 343 commits intoNCAR:mainfrom
dustinswales:feature/containerized_ci
Open

Updates to CI testing + support for Intel oneApi.#658
dustinswales wants to merge 343 commits intoNCAR:mainfrom
dustinswales:feature/containerized_ci

Conversation

@dustinswales
Copy link
Member

@dustinswales dustinswales commented Feb 11, 2026

This PR contains several changes to the CI based testing system.

  • Replaces Ubuntu GitHub runner based CI with Containerized CI (Thanks @RobertPincus for paving the way in rte-rrtmgp!)
  • New Docker files for each piece of the CCPP SCM software stack (e.g. gnu, oneapi, netcdf).
  • Docker images (GNU, Intel oneAPI, nvhpc) for CCPP SCM
  • Intel oneAPI CI tests
  • Changes to utility scripts used during CI
  • Remove deprecated CI scripts

Other changes worth noting.

  • All test configurations are controlled in a single file, test/rt_test_cases.py. Individual tests can now be turned on/off.
  • Tests have been broken down into "supported", "legacy" and "dev". UPDATE: This is a workaround for the limitation on the number of SDFs that can be used with Intel oneAPI.

@grantfirl @scrasmussen @mkavulich @hertneky The failing tests (pushing image to dockerhub) are related to permissions. These tests all pass in my fork.

I was able to create a GitHub secret with my dockerhub credentials, but this didn't work. I also tried using the existing GitHub secrets for the dtcenter dockerhub repository, but this didn't work correctly either.
I don't have Admin access in this repo ?!? so maybe it's related to that?

@dustinswales dustinswales marked this pull request as ready for review February 11, 2026 21:50
@dustinswales
Copy link
Member Author

dustinswales commented Feb 25, 2026

@grantfirl @scrasmussen @hertneky
This is ready for review when you get a chance.
The new CI test to create the images is working, but it relies on github secrets, which can't be referenced by the incoming PR for security reasons.

Copy link
Collaborator

@hertneky hertneky 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, thanks!

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.

3 participants