Skip to content

Fix 5.x: Add git push permissions to release workflow - #1710

Closed
ricardozanini wants to merge 1 commit into
5.xfrom
fix-5x-workflow-permissions
Closed

ricardozanini wants to merge 1 commit into
5.xfrom
fix-5x-workflow-permissions

Conversation

@ricardozanini

Copy link
Copy Markdown
Collaborator

Fixes the permission error when pushing release tags and commits.

Problem

The release workflow fails at the git push step with:

remote: Permission to open-workflow-specification/sdk-java.git denied to github-actions[bot].
fatal: unable to access: The requested URL returned error: 403

Solution

  • Add permissions: contents: write at the workflow level
  • Explicitly pass token: ${{ secrets.GITHUB_TOKEN }} to the checkout action

This allows the GitHub Actions bot to push commits and tags back to the repository.

Testing

After merge, the next release should successfully push tags and version bump commits.

The workflow needs contents: write permission to push tags and commits.
This fixes the "Permission denied" error when pushing release tags.

Signed-off-by: GitHub Action <action@github.com>
Copilot AI lite review requested due to automatic review settings September 21, 2026 17:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

No unresolved review issues were identified.

Review effort: Lite
Findings: None

What changed in this PR

Updates the release workflow so GitHub Actions can push release tags and commits.

Changes:

  • Grants contents: write permission.
  • Passes GITHUB_TOKEN explicitly to checkout.
File Description
.github/​workflows/​release.yml Enables authenticated release pushes.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@ricardozanini

Copy link
Copy Markdown
Collaborator Author

Closing - adding this fix to PR #1702 instead

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.

3 participants