Skip to content

Fix pr-review workflow permissions: grant issues:write for failure tracking - #1920

Merged
v1v merged 1 commit into
mainfrom
copilot/fix-workflow-issues-again
Aug 27, 2026
Merged

v1v merged 1 commit into
mainfrom
copilot/fix-workflow-issues-again

Conversation

Copilot AI commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Problem

Workflow validation failed for trigger-pr-review.yml:

Error calling workflow 'elastic/ai-github-actions/.github/workflows/gh-aw-pr-review.lock.yml@aadcfd18c43d3f03138e53a725b3b176ce1448ba'. 
The nested job 'conclusion' is requesting 'issues: write', but is only allowed 'issues: read'.
The nested job 'safe_outputs' is requesting 'issues: write', but is only allowed 'issues: read'.

Root Cause

The compiled lock file (v0.87.4) includes conclusion and safe_outputs jobs that require issues: write to create failure tracking issues (via the report-failure-as-issue feature enabled by default). The trigger file only granted issues: read.

Changes

  • gh-agent-workflows/pr-review/example.yml: Changed issues: readissues: write with updated comment
  • .github/workflows/trigger-pr-review.yml: Regenerated from example.yml via make compile

The fix aligns caller permissions with the lock file's internal job requirements.

…acking

Co-authored-by: v1v <2871786+v1v@users.noreply.github.com>
@v1v
v1v marked this pull request as ready for review August 27, 2026 11:15
@v1v
v1v merged commit f22f2a7 into main Aug 27, 2026
17 checks passed
@v1v
v1v deleted the copilot/fix-workflow-issues-again branch August 27, 2026 11:17
@github-actions github-actions Bot added the small_boom Small PR blast radius; usually low human-review need label Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

small_boom Small PR blast radius; usually low human-review need

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants