Skip to content

build-ci v3: Update Infra For spack v1#353

Merged
CodeGat merged 6 commits intoapi-v2from
infra-update-v3
Dec 17, 2025
Merged

build-ci v3: Update Infra For spack v1#353
CodeGat merged 6 commits intoapi-v2from
infra-update-v3

Conversation

@CodeGat
Copy link
Copy Markdown
Member

@CodeGat CodeGat commented Nov 17, 2025

References issue ACCESS-NRI/build-ci#231 and PR ACCESS-NRI/build-ci#253
References project Spack `0.22` to `1.0` Migration

Important

This PR is a major update to the infrastructure. See below for the prerequisites for this repository to be able to merge this PR.

Important

This major version change marks the end of major infrastructure updates for CI using spack < 1.0. They will still get bug fixes and non-breaking features if required.
If you want to deploy to instances of spack < 1.0, you must use build-ci v2.
If you want to deploy to instances of spack >= 1.0, you must use build-ci v3.

Background

We are looking to transition build-ci to v3! This is due to a migration to spack v1.X, from v0.X, which offers many bug fixes, features, and optimisations.

One of these changes is the splitting of spacks core codebase from the builtin spack-packages repository, which means that we have another lever to tweak for our builds: the version of spack, spack-config, our spack-packages and the (new!) builtin spack-packages.

Since we want to leave open the possibility of forking the builtin spack-packages, we have also decided to rename ACCESS-NRI/spack-packages to ACCESS-NRI/access-spack-packages. See #295 for more info.

Therefore this is a major version update, as there are the following changes to the inputs (and analogous changes to the workflow outputs):

  • Deleted the optional spack-packages-ref input, which defaulted to main (in future, api-v1).
  • Added the optional access-spack-packages-ref input, which defaults to the api-v2 branch.
  • Added the optional builtin-spack-packages-ref input, which defaults to the main branch.

This means that:

  • If you want to use container-image-version: :rocky-0.22-*, or spack-ref < releases/1.0, you must stay on v2 due to the input changes.
  • If you want to use container-image-version: :rocky-1.*-*, or spack-ref >= releases/1.0, you must migrate to v3 due to the input changes.

It is possible to test across these versions of spack if required, via a mix of workflow versions. Ask @CodeGat if you need this functionality.

Features

The main new features include:

  • Proper Spack v1 Support: Users now can tweak both the builtin spack-packages repository, as well as our own access-spack-packages repository. All of the bugfixes, features and optimisations added since 0.22 are available to us for future inclusion into build-ci.

Prerequisites for Merging

  • Update build-ci entrypoints to v3 (this PR!)
  • Make sure that all SPRs work with this v3 infra

@CodeGat CodeGat self-assigned this Nov 17, 2025
@CodeGat CodeGat changed the base branch from main to api-v2 November 17, 2025 03:53
@harshula harshula force-pushed the api-v2 branch 3 times, most recently from c163f27 to f7a4cb5 Compare December 1, 2025 22:55
@harshula harshula force-pushed the api-v2 branch 8 times, most recently from 4878de1 to 266d395 Compare December 7, 2025 23:16
@CodeGat CodeGat marked this pull request as ready for review December 9, 2025 00:38
@CodeGat CodeGat requested a review from harshula December 9, 2025 00:42
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml Outdated
@CodeGat CodeGat requested a review from harshula December 11, 2025 03:24
@CodeGat
Copy link
Copy Markdown
Member Author

CodeGat commented Dec 11, 2025

@harshula I have updated this workflow to use the defaults from the called workflow by reading the defaults from the workflow itself. It's ready for review now!

@harshula harshula force-pushed the api-v2 branch 4 times, most recently from 2849be1 to 9c55156 Compare December 16, 2025 22:35
@harshula harshula force-pushed the api-v2 branch 5 times, most recently from 58dedf3 to f571b78 Compare December 17, 2025 00:58
@CodeGat CodeGat force-pushed the infra-update-v3 branch 2 times, most recently from b843d1e to 93506d8 Compare December 17, 2025 05:51
Copy link
Copy Markdown
Collaborator

@harshula harshula left a comment

Choose a reason for hiding this comment

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

🤞

@CodeGat CodeGat merged commit d5c0093 into api-v2 Dec 17, 2025
@CodeGat CodeGat deleted the infra-update-v3 branch December 17, 2025 06:29
harshula pushed a commit that referenced this pull request Dec 19, 2025
infra: Update to v3, get default spack-config-ref/spack-ref from build-ci workflow, use default container-image-version
harshula pushed a commit that referenced this pull request Jan 28, 2026
infra: Update to v3, get default spack-config-ref/spack-ref from build-ci workflow, use default container-image-version
harshula pushed a commit that referenced this pull request Feb 6, 2026
infra: Update to v3, get default spack-config-ref/spack-ref from build-ci workflow, use default container-image-version
harshula pushed a commit that referenced this pull request Feb 6, 2026
infra: Update to v3, get default spack-config-ref/spack-ref from build-ci workflow, use default container-image-version
harshula pushed a commit that referenced this pull request Feb 9, 2026
infra: Update to v3, get default spack-config-ref/spack-ref from build-ci workflow, use default container-image-version
harshula pushed a commit that referenced this pull request Feb 10, 2026
infra: Update to v3, get default spack-config-ref/spack-ref from build-ci workflow, use default container-image-version
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