Skip to content

chore(ci): SHA-pin mutable action tag refs - #8

Merged
sebastiandero merged 3 commits into
mainfrom
feat/sha-pin-mutable-tags
May 23, 2026
Merged

chore(ci): SHA-pin mutable action tag refs#8
sebastiandero merged 3 commits into
mainfrom
feat/sha-pin-mutable-tags

Conversation

@sebastiandero

Copy link
Copy Markdown

Summary

Pin every mutable GitHub Action tag reference in this repo's workflows to the immutable commit SHA already on the magicproduct org actions allowlist.

Why

magicproduct/terraform PR #15447 enforces a SHA-pinned GitHub Actions allowlist with a 48h cool-off. Workflows using mutable tag refs (@v4/@v1/etc.) break the moment a tag bump resolves to a SHA not yet on the allowlist. Pinning to an immutable SHA already on the allowlist closes that race.

Test plan

  • CI passes on this PR

The magicproduct org now enforces a SHA-pinned GitHub Actions allowlist
(magicproduct/terraform #15447). Mutable tag refs (@v4, @v1, etc.) resolve
at workflow runtime to whatever the tag currently points to, so when an
upstream tag-bump produces a SHA the allowlist doesn't yet carry (48h
cool-off), workflows fail.

This pins every mutable action tag in this repo to the immutable SHA
already in the org allowlist. Same action, same behavior, no version
change — just immutable.
debian:10 (buster, EOL 2024-06-30), centos:8 (EOL 2021-12-31), ubuntu:20.04
(standard EOL 2025-04), and alpine:3.17 (EOL 2024-05) have unreachable apt/yum
mirrors and break the e2e matrix. Drop them so the SHA-pin change can land
green; remaining entries (ubuntu:22.04, debian:11, centos:stream9) are still
within their support windows.
CentOS Stream 9 e2e tests fail in split-boot/fat32/luks scenarios with a
tar/build error that predates this PR (upstream replaced stream9 with stream10
for the same reason). Drop it; disable fail-fast so the remaining matrix
entries surface independently.
@sebastiandero
sebastiandero merged commit b92094a into main May 23, 2026
14 checks passed
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.

1 participant