Merge pull request #109 from nb-ohad/rbac-fix

ClientProfile Controller: add missing CephConnection delete RBAC
ceph · Aug 8, 2024 · 1efdddd · 1efdddd
2 parents deb792a + dbe3397
commit 1efdddd
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/config/rbac/role.yaml b/config/rbac/role.yaml
@@ -57,6 +57,7 @@ rules:
   resources:
   - cephconnections
   verbs:
+  - delete
   - get
   - list
   - update

diff --git a/internal/controller/clientprofile_controller.go b/internal/controller/clientprofile_controller.go
@@ -40,7 +40,7 @@ import (
 //+kubebuilder:rbac:groups=csi.ceph.io,resources=clientprofiles,verbs=get;list;watch;create;update;patch;delete
 //+kubebuilder:rbac:groups=csi.ceph.io,resources=clientprofiles/status,verbs=get;update;patch
 //+kubebuilder:rbac:groups=csi.ceph.io,resources=clientprofiles/finalizers,verbs=update
-//+kubebuilder:rbac:groups=csi.ceph.io,resources=cephconnections,verbs=get;list;watch;update
+//+kubebuilder:rbac:groups=csi.ceph.io,resources=cephconnections,verbs=get;list;watch;update;delete
 //+kubebuilder:rbac:groups="",resources=configmaps,verbs=get;list;watch;create;update;patch;delete
 
 // ClientProfileReconciler reconciles a ClientProfile object
-Original file line number
+Diff line change
@@ Expand Up / @@ -57,6 +57,7 @@ rules: @@
       resources:
       - cephconnections
       verbs:
+      - delete
       - get
       - list
       - update
@@ Expand Down @@