Skip to content

Infra Update: New Release Provenance Database and Workflow Triggers#105

Merged
CodeGat merged 1 commit intomainfrom
infra-update-v5
May 16, 2025
Merged

Infra Update: New Release Provenance Database and Workflow Triggers#105
CodeGat merged 1 commit intomainfrom
infra-update-v5

Conversation

@CodeGat
Copy link
Copy Markdown
Member

@CodeGat CodeGat commented May 7, 2025

References issue ACCESS-NRI/build-cd#263 and PR ACCESS-NRI/build-cd#269
References issue ACCESS-NRI/build-cd#238

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 Infrastructure update moves away from the old build database at https://experiment-metadb-seven.vercel.app/release-provenance to a more modern database as part of ACCESS-NRI/tracking_services, at https://reporting-dev.access-nri-store.cloud.edu.au/release-provenance/releases.

We also allow rebuilding of prereleases on.pull_request.types.ready_for_review and on.pull_request.types.converted_to_draft, which removes a small but significant loophole in which someone works on a draft PR (with less restrictive spack.yaml checks) and then converts it to Ready For Review.

Prerequisites for Merging

  • Update build-cd entrypoints (this PR!)
  • Update triggers (also this PR!)
  • Delete Repo Secret secrets.BUILD_DB_CONNECTION_STR (from the old build database)
  • Add GitHub Environment (* Release) Variable vars.TRACKING_SERVICES_POST_URL (URL for posting release data)
  • Add GitHub Environment (* Release) Secret secrets.TRACKING_SERVICES_POST_TOKEN (Token for posting release data)

@CodeGat CodeGat added the type:infra Related to CI/CD label May 7, 2025
@CodeGat CodeGat self-assigned this May 7, 2025
@atteggiani atteggiani self-requested a review May 16, 2025 01:39
Copy link
Copy Markdown

@atteggiani atteggiani 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 marked this pull request as ready for review May 16, 2025 05:20
@CodeGat CodeGat merged commit f3f9b5a into main May 16, 2025
@CodeGat CodeGat deleted the infra-update-v5 branch May 16, 2025 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:infra Related to CI/CD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants