Skip to content

Replace GitHub secrets with variables in release workflow - #261

Merged
carlosmmatos merged 2 commits into
mainfrom
atlas/task-122
Jul 8, 2026
Merged

Replace GitHub secrets with variables in release workflow#261
carlosmmatos merged 2 commits into
mainfrom
atlas/task-122

Conversation

@carlosmmatos

Copy link
Copy Markdown
Contributor

The release workflow previously authenticated to Google Cloud and configured image publishing using GitHub secrets. Since we can no longer use GitHub secrets, this migrates those values to GitHub repository variables.

All of the values involved are non-sensitive public identifiers — the workload identity provider path, the service account email, image registry and names, and the marketplace service annotation — so they are safe to store as plaintext variables. Authentication to GCP continues to use Workload Identity Federation over the GitHub OIDC token, which mints a short-lived access token at runtime, so no long-lived credentials are stored anywhere. The gcr.io docker login now uses the fixed oauth2accesstoken username with that runtime token instead of a stored registry login secret.

The container and deployer jobs are unchanged in behavior; only their input sources moved from secrets.* to vars.*.

Replace secrets.WORKLOAD_IDENTITY_PROVIDER, secrets.SERVICE_ACCOUNT, and
secrets.SERVICE_ANNOTATION with repository variables (vars.*), and inline the
gcr.io login username as the well-known oauth2accesstoken value. None of these
are secret: the WIF provider path and service account email are keyless-auth
identifiers, the annotation is OCI image metadata, and oauth2accesstoken is the
canonical username used with a short-lived GCP access token.

Also switch the release trigger from types:[created] to types:[published] so
releases published from a draft fire the workflow.
@carlosmmatos
carlosmmatos merged commit 93aea71 into main Jul 8, 2026
2 checks passed
@carlosmmatos
carlosmmatos deleted the atlas/task-122 branch July 8, 2026 18:14
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.

4 participants