Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deployment Infrastructure V4: Multi-HPC Deployments #20

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

CodeGat
Copy link
Member

@CodeGat CodeGat commented Jan 29, 2025

References issue ACCESS-NRI/build-cd#121 and detailed PR ACCESS-NRI/build-cd#218

Important

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.

Background

This update to the deployment infrastructure allows deployment to multiple different HPC systems (or the same one but a different spack instance) in parallel.

Some of the major changes relevant to a model deployment repository include:

Prerequisites for Merging

  • Add values for repo-level vars.[PRE]RELEASE_DEPLOYMENT_TARGETS
  • Update repo-level vars.SPACK_YAML_SCHEMA_VERSION to 1-0-4
  • Update entrypoints to @v4, update ci.yml pr-closed jobs model input to root-sbd, update cd.yml cd job permissions.pull-requests:write (This PR!)
  • Add GitHub Environment for Releases with name SUPERCOMPUTER Release rather than SUPERCOMPUTER

@CodeGat CodeGat self-assigned this Jan 29, 2025
@CodeGat CodeGat marked this pull request as ready for review February 4, 2025 22:46
Copy link
Member

@aidanheerdegen aidanheerdegen left a comment

Choose a reason for hiding this comment

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

LGTM

@CodeGat CodeGat merged commit d6851a2 into main Feb 5, 2025
@CodeGat CodeGat deleted the infra-update-v4 branch February 5, 2025 03:16
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