1Password / connect-helm-charts

Official 1Password Helm Charts
https://developer.1password.com
MIT License
93 stars 74 forks source link

Create RBAC resources by default when operator is enabled #37

Closed florisvdg closed 3 years ago

florisvdg commented 3 years ago

Previously, to get an out-of-the-box working operator, you'd have to add 4 flags: --set operator.create=true --set operator.clusterRole.create=true --set operator.serviceAccount.create=true --set operator.roleBinding.create=true.

This PR changes this behavior by taking whatever value the operator.create flag has as the default value of the RBAC resource creation flags.

So now, you only have to add --set operator.create=true and you get the correct RBAC setup. The resources can still individually be disabled by explicitly setting them to false.