Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Helm chart 2.0.2 seems to lacks list verb for secrets in cluster role #708

Closed
cmeury opened this issue Jan 12, 2022 · 1 comment · Fixed by #715
Closed

Helm chart 2.0.2 seems to lacks list verb for secrets in cluster role #708

cmeury opened this issue Jan 12, 2022 · 1 comment · Fixed by #715
Assignees
Labels

Comments

@cmeury
Copy link

cmeury commented Jan 12, 2022

Which component: Helm Chart

Describe the bug
The helm chart version 2.0.2 contains a cluster role with the following rule, among others:

  - apiGroups:
      - ""
    resources:
      - secrets
    verbs:
      - get
      - create
      - update
      - delete

On a Kubernetes cluster with version v1.21.5, this is seemingly not enough, as the controller states upon start:

panic: secrets is forbidden: User "system:serviceaccount:kube-system:sealed-secrets" cannot list resource "secrets" in API group "" in the namespace "kube-system"

goroutine 1 [running]:
main.main()
	/home/runner/work/sealed-secrets/sealed-secrets/cmd/controller/main.go:271 +0x225

Adding list to the verbs resolves the issue.

Additional context
Curiously enough, it looks like this popped before: helm/charts#15837

@juan131 juan131 added the bug label Jan 20, 2022
@juan131 juan131 self-assigned this Jan 20, 2022
@juan131
Copy link
Collaborator

juan131 commented Jan 20, 2022

Thanks for reporting it @cmeury ! I'll send a PR to address a couple of RBAC issues we recently discovered.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants