Skip to content

fix(ci): missing autolabeler permissions#1608

Merged
jvanz merged 1 commit intokubewarden:mainfrom
jvanz:fix-autolabeler-permissions
Mar 26, 2026
Merged

fix(ci): missing autolabeler permissions#1608
jvanz merged 1 commit intokubewarden:mainfrom
jvanz:fix-autolabeler-permissions

Conversation

@jvanz
Copy link
Copy Markdown
Member

@jvanz jvanz commented Mar 26, 2026

Description

Adds the missing permissions to allow autolabeler to add labels in the pull requests.

Adds the missing permissions to allow autolabeler to add labels in the
pull requests.

Signed-off-by: José Guilherme Vanz <jguilhermevanz@suse.com>
@jvanz jvanz self-assigned this Mar 26, 2026
Copilot AI review requested due to automatic review settings March 26, 2026 12:40
@jvanz jvanz requested a review from a team as a code owner March 26, 2026 12:40
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the GitHub Actions autolabeler workflow to grant write permissions needed for applying labels to pull requests.

Changes:

  • Add pull-requests: write to workflow-level permissions in the autolabeler workflow.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 8 to 11
permissions:
contents: read
pull-requests: write

Copy link

Copilot AI Mar 26, 2026

Choose a reason for hiding this comment

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

The job already sets permissions, which (per GitHub Actions semantics) overrides the workflow-level permissions. As written, the autolabeler job will only have pull-requests: write and will not inherit contents: read, and adding pull-requests: write at the workflow level will not change the token permissions inside the job. Consider either removing the job-level permissions block (so the job inherits contents: read + pull-requests: write), or explicitly including both permissions at the job level.

Copilot uses AI. Check for mistakes.
@jvanz jvanz merged commit cbc2966 into kubewarden:main Mar 26, 2026
30 of 31 checks passed
@github-project-automation github-project-automation Bot moved this from Pending review to Done in Kubewarden Admission Controller Mar 26, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.44%. Comparing base (f7ba826) to head (60048bb).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1608      +/-   ##
==========================================
+ Coverage   80.42%   80.44%   +0.01%     
==========================================
  Files         127      127              
  Lines       16411    16411              
==========================================
+ Hits        13199    13202       +3     
+ Misses       3212     3209       -3     
Flag Coverage Δ
rust-tests 80.44% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

jvanz added a commit to jvanz/adm-controller that referenced this pull request Mar 26, 2026
…-permissions"

This reverts commit cbc2966, reversing
changes made to f7ba826.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants