Skip to content

Comments

Reduce Trivy Operator permissions#1224

Merged
jakub-racek-swi merged 7 commits intomasterfrom
trivy-permissions
Dec 10, 2025
Merged

Reduce Trivy Operator permissions#1224
jakub-racek-swi merged 7 commits intomasterfrom
trivy-permissions

Conversation

@jakub-racek-swi
Copy link
Contributor

@jakub-racek-swi jakub-racek-swi commented Dec 4, 2025

This PR will limit the permissions given to Trivy operator to reflect the only feature we use, that is image vulnerability scanning.

I have disabled access to secrets stored outside the Trivy namespace. To enable scanning of images stored in private registries, the user is able to grant the access rights to Trivy through either:

  • configuring those secrets manually (privateRegistryScanSecretsNames setting)
  • linking the trivy service account to the corresponding IAM role (this is outside the scope of the chart, documented in values)

@github-actions github-actions bot added the helm label Dec 4, 2025
@jakub-racek-swi jakub-racek-swi marked this pull request as ready for review December 4, 2025 14:08
@jakub-racek-swi jakub-racek-swi requested a review from a team as a code owner December 4, 2025 14:08
@etichy etichy self-assigned this Dec 8, 2025
gantrior
gantrior previously approved these changes Dec 9, 2025
Copy link
Contributor

@etichy etichy left a comment

Choose a reason for hiding this comment

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

LGTM -

Co-authored-by: Eduard Tichy <17002066+etichy@users.noreply.github.com>
@jakub-racek-swi jakub-racek-swi merged commit 5b68c48 into master Dec 10, 2025
27 of 28 checks passed
@jakub-racek-swi jakub-racek-swi deleted the trivy-permissions branch December 10, 2025 13:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

3 participants