8000 ci: push release tags with the automation token by ctreatma · Pull Request #357 · equinix/pulumi-equinix · GitHub
[go: up one dir, main page]

Skip to content

ci: push release tags with the automation token - #357

Merged
ctreatma merged 1 commit into
mainfrom
fix-tag-release-token
Oct 1, 2026
Merged

ctreatma merged 1 commit into
mainfrom
fix-tag-release-token

Conversation

@ctreatma
@ctreatma ctreatma commented Oct 1, 2026
Copy link
Copy Markdown
Contributor

Summary

The equinix-tag-release workflow created the v0.33.0 tag, but the tag push didn't trigger release.yml.

actions/checkout persists GITHUB_TOKEN (default persist-credentials: true) as an http.https://github.com/.extraheader AUTHORIZATION header, and git sends that header instead of the credentials in the push URL. So the tag was pushed with GITHUB_TOKEN rather than PULUMI_PROVIDER_AUTOMATION_TOKEN, and events triggered by GITHUB_TOKEN don't start other workflows.

This sets persist-credentials: false on the checkout, so the push authenticates with the automation token from the push URL. semantic-release's dry run is unaffected: it builds its own authenticated URL from the GITHUB_TOKEN in its environment. It also keeps the automation token out of the git config, so the semantic-release step (which installs packages from npm at runtime) can't read it.

After merging

release.yml only runs on tag pushes, so v0.33.0 needs to be pushed again to trigger it: either delete the remote tag and push it from a user account, or delete it and re-run equinix-tag-release.

🤖 Generated with Claude Code

actions/checkout persists GITHUB_TOKEN as an http.extraheader, and git
sends that header instead of the credentials in the tag push URL. The
v0.33.0 tag was therefore pushed with GITHUB_TOKEN, and pushes made with
GITHUB_TOKEN don't trigger other workflows, so release.yml never ran.

Don't persist credentials on checkout, so the tag push authenticates
with PULUMI_PROVIDER_AUTOMATION_TOKEN. semantic-release builds its own
authenticated URL from GITHUB_TOKEN, so it doesn't need them.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@ctreatma
ctreatma requested a review from a team October 1, 2026 19:27
@github-actions
github-actions Bot commented Oct 1, 2026
Copy link
Copy Markdown

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

Maintainer note: consult the runbook for dealing with any breaking changes.

@ctreatma
ctreatma merged commit 301489d into main Oct 1, 2026
21 checks passed
@ctreatma
ctreatma deleted the fix-tag-release-token branch October 1, 2026 19:41
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

0