Conversation
785d31c to
63c29d0
Compare
|
Note: Rebased all the changes from the |
9966fac to
9fb51d0
Compare
|
🚀 Attempted to deploy 🖥️
|
|
Although it installed fine, it failed to get the metadata from the restricted directory. I'll fix up this bug now. Edit: Fixed via ACCESS-NRI/build-cd#357 |
|
🚀 Attempted to deploy 🖥️
|
|
🚀 Attempted to deploy 🖥️
|
|
The above deployment was successful (as was the metadata generation). |
|
!update-configs |
|
🔧 Opening Model Configuration PRs in Configurations RequestedConfigurations requested from profile
Pull Requests OpenedThe following PRs were opened:
More details can be found in the workflow run: https://github.com/ACCESS-NRI/ACCESS-ESM1.6/actions/runs/21974705756 |
|
🚀 Attempted to deploy 🖥️
|
|
!update-configs |
|
🔧 Opening/Updating Model Configuration PRs in Configurations RequestedConfigurations requested from profile
Pull Requests Opened/UpdatedThe following PRs were opened/updated:
More details can be found in the workflow run: https://github.com/ACCESS-NRI/ACCESS-ESM1.6/actions/runs/22122642995 |
blimlim
left a comment
There was a problem hiding this comment.
Less familiar with the CI/CD side, but the spack.yaml looks good to me!
manodeep
left a comment
There was a problem hiding this comment.
I am not across the CI/CD bits, but everything looks fine.
Only thing might be to see if you could separate out the white-space-only changes in a different PR?
|
Thanks @manodeep ! We might need for it to be in this one just so it's compatible with the formatter in spack v1.1 |
|
Extract from the concretisation: We should try setting
|
|
Yeah Peter was confused as well. Maybe we should set the target explicitly for |
|
🚀 Attempted to deploy 🖥️
|
|
Seems like spack enforces targets are the same or more flexible for dependencies then ? |
|
I guess I should drop that change, then |
|
I guess with are stuck with the spack set targets for those packages |
|
Okay it's like nothing ever happened 😆 approve away |
References issue ACCESS-NRI/build-cd#313 and PR ACCESS-NRI/build-cd#326
References rollout issue ACCESS-NRI/build-cd#328
References project Spack `0.22` to `1.0` Migration
Note
This infrastructure update created a new Release, but it maintained reproducibility with the current
mainImportant
This PR is a major update to the deployment 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 deployments to
spack < 1.0.If you want to deploy to instances of
spack < 1.0, you must usebuild-cd < v8.If you want to deploy to instances of
spack >= 1.0, you must usebuild-cd >= v8.Background
We are moving on up to
spack v1! This update to spack contains many bug fixes, optimisations and new features that can be incorporated intobuild-cd.This update also contains changes that make
spack v0.Xincompatible with thisbuild-cd v8update - the most prominent one being the splitting of spacks core codebase from it's builtin spack-packages repo. This means that for provenance, one to keep track of bothbuiltinspack-packages and our own, renamedaccess-spack-packages.build-cdwill centrally control the version of builtinspack-packagesfor a given instance, andconfig/versions.jsonsspack-packageskey is renamed toaccess-spack-packages.As noted earlier, this major version of
build-cdcan only supportspack v1, due tospack v1-specific commands in the infrastructure. If you still want to deploy tospack v0.X, you will need to changebuild-cdtov7, and update theconfig/versions.jsonfile/spack.yaml.Features
spack.config!: This is now handled inspack-configvia the.custom-scopeskey inconfig/versions.json!Prerequisites for Merging
build-cdentrypoints (this PR!)config/versions.jsonwith new inputs (also this PR!)spack v1🚀 The latest prerelease
access-esm1p6/pr168-6at 97f59c0 is here: #168 (comment) 🚀🚀 The latest prerelease
access-esm1p6/pr168-7at 34f23bd is here: #168 (comment) 🚀