Add K8up permissions to clusterrole

ccremer commented 2 years ago

Summary

[x] Categorize the PR by setting a good title and adding one of the labels: bug, enhancement, documentation, change, breaking, dependency as they show up in the changelog
[x] PR contains the label area:chart
[x] PR contains the chart label, e.g. chart:provider-postgresql
[x] Variables are documented in the values.yaml using the format required by Helm-Docs.
[x] Chart Version bumped if immediate release after merging is planned
[x] I have run make chart-docs
[x] Link this PR to related code release or other issues.

zugao commented 2 years ago

Can you also add pods and PVCs in this PR? Same permissions as Schedules

ccremer commented 2 years ago

Can you also add pods and PVCs in this PR? Same permissions as Schedules

That requires the generator to be updated as well, which would be a code change that can't go into this PR. If you create a PR that adds those permissions, I'm happy to rebase this PR on your PR. See https://github.com/vshn/appcat-service-postgresql/blob/d3404861b869c27f4970913f6ee4e8720117e78f/operator/standalone/controller.go#L28-L39

zugao commented 2 years ago

Lets not get confusing with this, lets merge it