Skip to content

ci(release): grant the mint token workflows write - #5

Merged
mattwilkinsonn merged 1 commit into
rigel-releasefrom
upstream-woodpecker/rig-5434-mint-workflows-write
Oct 16, 2026
Merged

mattwilkinsonn merged 1 commit into
rigel-releasefrom
upstream-woodpecker/rig-5434-mint-workflows-write

Conversation

@rigel-mintaka

@rigel-mintaka rigel-mintaka commented Oct 15, 2026 •

Copy link
Copy Markdown

Why

Matt ruled option A on RIG-5434. GitHub returns 403 on tag creation when the target commit's workflow tree matches no branch tip, unless the token has workflows: write. The nix2container mint (RIG-5346) needs this. Same change as RigelBuild/woodpecker#100 and RigelBuild/jj-vine#38.

What

  • .github/workflows/release-tag.yaml: the mint App token adds permission-workflows: write next to permission-contents: write.
  • .github/scripts/release-tag.test.sh: asserts both permissions are write, and pins the exact permission set to permission-contents,permission-workflows.

Verification

  • Red: the test failed with FAIL mint token has contents and workflows write only before the workflow change.
  • Green: the test passes after it, with 0 FAIL lines. actionlint and shellcheck pass.

Dependency

The automation App has workflows=write (approved on RIG-5057). Installation acceptance is tracked on RIG-5434.

Refs RIG-5434

GitHub returns 403 on tag creation when the target commit's workflow
tree matches no branch tip, unless the token has workflows write. Add it
next to contents write, and pin the exact permission set in the release
tag test.

Refs RIG-5434

Co-authored-by: Matt Wilkinson <matt@rigel.build>
@linear-code

linear-code Bot commented Oct 15, 2026

Copy link
Copy Markdown

RIG-5434

@rigel-mintaka rigel-mintaka changed the title fix(buildImage): exclude relocated copyToRoot paths from initialized nix DB ci(release): grant the mint token workflows write Oct 15, 2026
@rigel-mintaka
rigel-mintaka changed the base branch from main to rigel-release October 15, 2026 18:29
@rigel-mintaka
rigel-mintaka marked this pull request as ready for review October 15, 2026 18:45
@rigel-mintaka

Copy link
Copy Markdown
Author

Review loop: all-clear

  • reviewedSha: 13764853abe9882eacad8826a88e1dc6c39cf899
  • reviewedBase: rigel-release
  • Reviewer: single review agent (6-line workflow + test diff, single-agent eligible).
  • Findings: high 0, medium 0, low 0.
  • Checked: the hyphenated-key yq queries under mikefarah yq v4.53.3; removing permission-workflows makes the assertion fail; both values are asserted; the comment is 98 characters; the body matches the diff.
  • Head check: the current head and base equal the reviewed head and base.
  • CI: 6/6 check-runs success on this head, none pending.

@mattwilkinsonn
mattwilkinsonn merged commit 7663114 into rigel-release Oct 16, 2026
6 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.

2 participants