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

🌱 chore: differentiate ghcr and prod registry workflows #183

Merged
merged 1 commit into from
Oct 11, 2023

Conversation

salasberryfin
Copy link
Contributor

@salasberryfin salasberryfin commented Oct 3, 2023

What this PR does / why we need it:

The current release workflow builds and publishes images to both GHCR and Prime registry. While creating the first Rancher Turtles release candidate we thought we could split these actions into different jobs for each registry.

  • Building and publishing images in parallel reduces workflow time.
  • Easier to troubleshoot independently when failure occurs.

This change also narrows the scope of environment variables to the specific jobs.

Which issue(s) this PR fixes:
Fixes #169

Special notes for your reviewer:

Checklist:

  • squashed commits into logical changes
  • includes documentation
  • adds unit tests
  • adds or updates e2e tests

@salasberryfin salasberryfin requested a review from a team as a code owner October 3, 2023 15:14
@salasberryfin salasberryfin changed the title [WIP] 🌱 chore: differentiate ghcr and prod registry workflows 🌱 chore: differentiate ghcr and prod registry workflows Oct 4, 2023
@salasberryfin salasberryfin force-pushed the split-image-push-workflows branch from 5a04fb3 to 0be4d4a Compare October 4, 2023 09:29
@salasberryfin
Copy link
Contributor Author

This needs a major rebase after adding SLSA signing. I'm changing it back to WIP.

@salasberryfin salasberryfin changed the title 🌱 chore: differentiate ghcr and prod registry workflows [WIP] 🌱 chore: differentiate ghcr and prod registry workflows Oct 5, 2023
@salasberryfin salasberryfin force-pushed the split-image-push-workflows branch from 0be4d4a to b838949 Compare October 10, 2023 14:36
@salasberryfin salasberryfin force-pushed the split-image-push-workflows branch from b838949 to d1354f2 Compare October 10, 2023 14:46
@salasberryfin salasberryfin changed the title [WIP] 🌱 chore: differentiate ghcr and prod registry workflows 🌱 chore: differentiate ghcr and prod registry workflows Oct 10, 2023
@salasberryfin
Copy link
Contributor Author

salasberryfin commented Oct 10, 2023

The PR is now rebased and includes the SLSA signing feature merged last week, which is hopefully not affected by this. Additionally, added outputs for prod-images https://github.com/rancher-sandbox/rancher-turtles/blob/d1354f273d2e02643a5fccada0515c25cd02cde7/.github/workflows/release.yaml#L141 to the existing ones for ghcr-images.

Copy link
Contributor

@richardcase richardcase left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@richardcase richardcase merged commit f1bc424 into rancher:main Oct 11, 2023
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Differentiate workflows that push to Prime registry and GitHub registry
4 participants