Skip to content

fix(ci): grant E2E permissions to release workflows#2376

Merged
elezar merged 1 commit into
mainfrom
fix/release-dev-e2e-permissions/pimlock
Jul 20, 2026
Merged

fix(ci): grant E2E permissions to release workflows#2376
elezar merged 1 commit into
mainfrom
fix/release-dev-e2e-permissions/pimlock

Conversation

@pimlock

@pimlock pimlock commented Jul 20, 2026

Copy link
Copy Markdown
Collaborator

Summary

Restore Release Dev workflow startup by granting the reusable E2E workflow the token permissions it requests. Apply the same fix to Release Tag so the next tagged release does not hit the latent startup failure.

The regression was introduced by #2311, which added artifact downloads and actions: read to .github/workflows/e2e-test.yml but updated only the branch and merge-queue caller permissions. Release Dev has failed at startup since that merge: https://github.com/NVIDIA/OpenShell/actions/runs/29770548461

Related Issue

Follow-up to #2311. No issue.

Changes

  • Grant actions: read, contents: read, and packages: read to the Release Dev E2E caller
  • Grant the same least-privilege permissions to the identically affected Release Tag E2E caller
  • Leave merge-queue E2E configuration unchanged because it already grants the required permissions

Testing

  • mise run pre-commit passes
  • Unit tests added/updated (not applicable; workflow permissions only)
  • E2E tests added/updated (not applicable; workflow permissions only)
  • Parsed the modified workflow YAML and verified caller permissions satisfy the reusable workflow permissions
  • git diff --check passes

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (not applicable; workflow permissions only)

Signed-off-by: Piotr Mlocek <pmlocek@nvidia.com>
@github-actions

Copy link
Copy Markdown

@elezar
elezar added this pull request to the merge queue Jul 20, 2026
Merged via the queue into main with commit a9f7131 Jul 20, 2026
31 checks passed
@elezar
elezar deleted the fix/release-dev-e2e-permissions/pimlock branch July 20, 2026 21:17
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