Skip to content

fix: add actions:read permission to scheduled-test workflow#6

Merged
Arthurvdv merged 2 commits into
mainfrom
fix/scheduled-test-permissions
Apr 27, 2026
Merged

fix: add actions:read permission to scheduled-test workflow#6
Arthurvdv merged 2 commits into
mainfrom
fix/scheduled-test-permissions

Conversation

@Arthurvdv

Copy link
Copy Markdown
Member

Problem

The scheduled-test.yml workflow fails with:

The nested job 'report' is requesting 'actions: read', but is only allowed 'actions: none'.

The report job in build-test.yml declares permissions: { actions: read }, but the calling workflow (scheduled-test.yml) only granted contents: read and checks: write. Since reusable workflows inherit permissions from the caller, the report job was denied actions: read.

Fix

Add actions: read to the top-level permissions block in scheduled-test.yml, matching what build-and-release.yml already has.

Fixes: https://github.com/ALCops/mcp-server/actions/runs/24984185695

Arthurvdv and others added 2 commits April 24, 2026 18:11
The build-test.yml reusable workflow's 'report' job requires
'actions: read' to download artifacts via dorny/test-reporter.
When the caller (build-and-release.yml) sets explicit permissions
without 'actions', it defaults to 'none', blocking the nested job.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The report job in build-test.yml requires 'actions: read' permission,
but the scheduled-test caller workflow only granted 'contents: read'
and 'checks: write'. This caused the workflow to fail with:
'The nested job report is requesting actions: read, but is only
allowed actions: none.'

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@Arthurvdv Arthurvdv merged commit fd899ca into main Apr 27, 2026
9 checks passed
@Arthurvdv Arthurvdv deleted the fix/scheduled-test-permissions branch April 27, 2026 09:03
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.

1 participant