Skip to content

Add contents read permission for release chart#564

Merged
simonmarty merged 1 commit intomainfrom
perm
Feb 4, 2026
Merged

Add contents read permission for release chart#564
simonmarty merged 1 commit intomainfrom
perm

Conversation

@i-am-SR
Copy link
Contributor

@i-am-SR i-am-SR commented Feb 3, 2026

Issue #, if available:

Description of changes:
Add contents read permission for release chart

The release chart workflow failed: https://github.com/aws/secrets-store-csi-driver-provider-aws/actions/runs/21648520700

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@i-am-SR i-am-SR requested a review from a team as a code owner February 3, 2026 21:32
@codecov
Copy link

codecov bot commented Feb 3, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 61.56%. Comparing base (479590a) to head (72f2533).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #564   +/-   ##
=======================================
  Coverage   61.56%   61.56%           
=======================================
  Files          11       11           
  Lines         752      752           
=======================================
  Hits          463      463           
  Misses        274      274           
  Partials       15       15           

☔ 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.

@i-am-SR i-am-SR added the safe-to-test Pull Request has been manually reviewed and deemed to be safe to run integration tests on. label Feb 3, 2026
@simonmarty simonmarty enabled auto-merge (squash) February 4, 2026 00:18
@simonmarty simonmarty disabled auto-merge February 4, 2026 00:19
@simonmarty simonmarty merged commit 73ae833 into main Feb 4, 2026
12 of 14 checks passed
@simonmarty simonmarty deleted the perm branch February 4, 2026 00:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

safe-to-test Pull Request has been manually reviewed and deemed to be safe to run integration tests on.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants