Skip to content

Add checks permission to flaky-tests workflow#414

Merged
soemo merged 2 commits intomainfrom
give-flaky-test-workflow-more-read-rights
Mar 16, 2026
Merged

Add checks permission to flaky-tests workflow#414
soemo merged 2 commits intomainfrom
give-flaky-test-workflow-more-read-rights

Conversation

@soemo
Copy link
Contributor

@soemo soemo commented Mar 16, 2026

Type of Change

  • Bugfix
  • Enhancement / new feature
  • Refactoring
  • Documentation

Description

Add permissions: checks: read to find-flaky-tests workflow

The workflow is passing ${{ secrets.GITHUB_TOKEN }} as the auth token to find_flaky_tests.py, but GITHUB_TOKEN on scheduled workflows only gets contents: read and metadata: read — not checks: read, which is required to call the GET /repos/.../commits/{sha}/check-suites GitHub API endpoint.

Checklist

  • Add relevant labels (for example type of change or patch/minor/major)
  • Make sure not to introduce some mistakes
  • Update documentation
  • Review the Contributing Guideline and sign CLA
  • Reference relevant issue(s) and close them after merging

@soemo soemo added the patch Pull request with little updates label Mar 16, 2026
@soemo soemo marked this pull request as ready for review March 16, 2026 17:54
@soemo soemo requested a review from a team as a code owner March 16, 2026 17:54
@soemo soemo requested review from flaxel and timkante March 16, 2026 17:54
@soemo soemo merged commit 963c984 into main Mar 16, 2026
8 checks passed
@soemo soemo deleted the give-flaky-test-workflow-more-read-rights branch March 16, 2026 18:15
@github-actions github-actions bot locked and limited conversation to collaborators Mar 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

patch Pull request with little updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants