Skip to content

ci: Add explicit permissions for release-please#61

Merged
aaron-zeisler merged 1 commit into
mainfrom
aaronz/SDK-2549/fix-release-please-permissions
Jul 22, 2026
Merged

ci: Add explicit permissions for release-please#61
aaron-zeisler merged 1 commit into
mainfrom
aaronz/SDK-2549/fix-release-please-permissions

Conversation

@aaron-zeisler

Copy link
Copy Markdown

Summary

  • Add a job-level permissions: { contents: write, pull-requests: write } block to .github/workflows/release-please.yml, mirroring the create-prs job in .github/workflows/check-go-versions.yml.

Background

The Run Release Please workflow started failing with a 403 after merging #60:

✖ Error when creating branch
✖ RequestError [HttpError]: Error creating Pull Request: Resource not accessible by integration
   status: 403

The job had no permissions: block, so its GITHUB_TOKEN was read-only under the (now-tightened) default, and release-please-action couldn't create the release branch or PR. This blocked cutting v1.11.1, which contains the stream-close deadlock fix from #60.

Without an explicit permissions block, the job's GITHUB_TOKEN is
read-only under the tightened default, so release-please-action fails
with 403 Resource not accessible by integration when creating the
release branch and PR. Mirrors the create-prs job in
check-go-versions.yml.
@aaron-zeisler aaron-zeisler changed the title fix(ci): grant contents/PR write permissions to release-please job ci: Add explicit permissions for release-please Jul 22, 2026
@aaron-zeisler
aaron-zeisler marked this pull request as ready for review July 22, 2026 16:44
@aaron-zeisler
aaron-zeisler requested a review from a team as a code owner July 22, 2026 16:44
@aaron-zeisler
aaron-zeisler merged commit 52e7854 into main Jul 22, 2026
9 checks passed
@aaron-zeisler
aaron-zeisler deleted the aaronz/SDK-2549/fix-release-please-permissions branch July 22, 2026 17:48
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.

2 participants