Skip to content

Containerize GCC & Intel CI workflows - #1158

Merged
BrianCurtis-NOAA merged 3 commits into
ufs-community:developfrom
AlexanderRichert-NOAA:ci-use-ctrs
May 28, 2026
Merged

Containerize GCC & Intel CI workflows#1158
BrianCurtis-NOAA merged 3 commits into
ufs-community:developfrom
AlexanderRichert-NOAA:ci-use-ctrs

Conversation

@AlexanderRichert-NOAA

@AlexanderRichert-NOAA AlexanderRichert-NOAA commented May 15, 2026

Copy link
Copy Markdown
Collaborator

DESCRIPTION OF CHANGES:

This PR containerizing the GCC and Intel CI workflows based on the containers generated by https://github.com/NOAA-EMC/ci-common-build-cache.

Note: I haven't looked yet at precaching test files, either in the UFS_UTILS container images or through the Actions cache.

TESTING

PR CI runs successfully with updated workflows (including CTest & installation).

DEPENDENCIES:

none

DOCUMENTATION:

Workflows are internally documented with comments, including links to https://github.com/NOAA-EMC/ci-common-build-cache documentation.

LINKED ISSUES:

@AlexanderRichert-NOAA

Copy link
Copy Markdown
Collaborator Author

@BrianCurtis-NOAA putting this here for discussion/proof of concept for containerizing workflows beyond NCEPLIBS.

@AlexanderRichert-NOAA

Copy link
Copy Markdown
Collaborator Author

In practice it looks like the repo-level caching for the oneAPI compilers only saves about 15-20 seconds, so, it may or may not be worth the cache space with respect to the 10GB limit.

@BrianCurtis-NOAA

Copy link
Copy Markdown
Collaborator

@AlexanderRichert-NOAA I assume this method allows for easy upgrading to newer versions as needed? Is there a way to tell the total space it's using in our 10GB limit?

@AlexanderRichert-NOAA

Copy link
Copy Markdown
Collaborator Author

That's the idea behind the commented block about modifying the Spack installation. We could try it out here if maybe you have an example of a package that gets updated frequently.

I don't think GitHub directly reports the total usage, but you can basically look under https://github.com/ufs-community/UFS_UTILS/actions/caches and add everything up (if this PR were merged, I think it would just be the Intel installation; conceivably in the future test files could get cached there as well, or in the container images).

@BrianCurtis-NOAA

Copy link
Copy Markdown
Collaborator

@AlexanderRichert-NOAA I think this is the way forward, do you need to check/do anything else before merging?

@BrianCurtis-NOAA
BrianCurtis-NOAA marked this pull request as ready for review May 28, 2026 16:42
@AlexanderRichert-NOAA

Copy link
Copy Markdown
Collaborator Author

No, I think this is good. If I don't hear from you sooner I'll check in in a month or so just to see how it's going and if there are any lingering issues.

@BrianCurtis-NOAA
BrianCurtis-NOAA merged commit 2fb8d3f into ufs-community:develop May 28, 2026
2 checks passed
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.

2 participants