fluxcd-community / helm-charts

Community maintained Helm charts for Flux
Apache License 2.0
121 stars 76 forks source link

fix: ensure resource name is all lowercase #143

Closed moritzjohner-form3 closed 1 year ago

moritzjohner-form3 commented 1 year ago

We're running with multitenancy.enabled=true and multitenancy.privileged=false. With 2.x.x we observe the following issue:

[...]  helm-controller  reconciliation failed: failed to get last release revision: query: failed to query with labels: 
    serviceaccounts "flux-admin" is forbidden: User "system:serviceaccount:flux-system:helm-controller" 
    cannot impersonate resource "serviceaccounts" in API group "" in the namespace "admin"

This is due to the role pointing to resource serviceAccount instead of serviceaccount.

What this PR does / why we need it:

Which issue this PR fixes

N/A

Special notes for your reviewer:

Checklist