search

openstack-k8s-operators / barbican-operator

Apache License 2.0
5 stars 24 forks source link

Add missing RBAC config #44

Closed dmendiza closed 11 months ago

dmendiza commented 11 months ago

This patch adds the missing RBAC config for BarbicanWorker and BarbicanKeystoneListner that is causing the controller to get stuck in a crash loop.

dmendiza commented 11 months ago

Thanks for the review, @xek . I had to do a bit of digging around in the operator sdk and kubebuilder docs to understand the annotations. IIUC, the annotations are used by make manifests to generate the role.yaml file. I added all the missing annotations to the Barbican controller and then updated the role.yaml file to use the one generated by the makefile. The other files are generated by the openstack-sdk tool when a new api is added, and I think they were left out by mistake in the patch that implemented BarbicanKeystoneListener.