Skip to content

fix(release): grant id-token: write on deploy job (unblocks OIDC)#19

Merged
asachs01 merged 1 commit into
mainfrom
fix/grant-id-token-write-to-deploy
May 13, 2026
Merged

fix(release): grant id-token: write on deploy job (unblocks OIDC)#19
asachs01 merged 1 commit into
mainfrom
fix/grant-id-token-write-to-deploy

Conversation

@asachs01
Copy link
Copy Markdown
Member

Last sweep's release runs all failed with:

The nested job 'deploy' is requesting 'id-token: write', but is only allowed 'id-token: none'.

The reusable deploy workflow declares id-token: write at its job level for Azure OIDC, but reusable workflows can only USE permissions the caller has GRANTED. My earlier refactor dropped the inline deploy job's permissions block; this PR restores it (job-scoped, not workflow-wide).

After merge, the next release should actually execute the deploy job and land on gwp-<vendor> by digest.

The reusable deploy workflow (wyre-technology/.github mcp-server-deploy.yml)
declares id-token: write at its job level for Azure OIDC. Reusable
workflows cannot request permissions the caller hasn't granted, so when
my earlier refactor dropped the original inline deploy job's
permissions block, the next release pipeline failed with:

  The nested job 'deploy' is requesting 'id-token: write',
  but is only allowed 'id-token: none'.

Grant id-token: write + contents: read on the deploy job (job-scoped,
not workflow-wide) so the called workflow can use OIDC to authenticate
to Azure.
Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

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

⚠️ Code review skipped — your organization's overage spend limit has been reached.

Code review is billed via overage credits. To resume reviews, an organization admin can raise the monthly limit at claude.ai/admin-settings/claude-code.

Once credits are available, push a new commit or reopen this pull request to trigger a review.

@asachs01 asachs01 merged commit b2a8185 into main May 13, 2026
8 of 10 checks passed
@asachs01 asachs01 deleted the fix/grant-id-token-write-to-deploy branch May 13, 2026 20:04
@github-project-automation github-project-automation Bot moved this from Todo to Done in MSP Claude Plugins May 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant