Merge pull request #2099 from pszczerbik/fix/2097

🐛 Add missing update permissions for HostUpdatePolicy CR to BMH controller
metal3-io · Dec 9, 2024 · ca8e696 · ca8e696
2 parents 43bc68f + 2f7c42e
commit ca8e696
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 1 deletion.
diff --git a/config/base/rbac/role.yaml b/config/base/rbac/role.yaml
@@ -74,4 +74,5 @@ rules:
   verbs:
   - get
   - list
+  - update
   - watch
diff --git a/config/render/capm3.yaml b/config/render/capm3.yaml
@@ -2431,6 +2431,7 @@ rules:
   verbs:
   - get
   - list
+  - update
   - watch
 ---
 apiVersion: rbac.authorization.k8s.io/v1

diff --git a/controllers/metal3.io/baremetalhost_controller.go b/controllers/metal3.io/baremetalhost_controller.go
@@ -103,7 +103,7 @@ func (info *reconcileInfo) publishEvent(reason, message string) {
 // +kubebuilder:rbac:groups=metal3.io,resources=dataimages/status,verbs=get;update;patch
 
 // Allow for updating hostupdatepolicies
-//+kubebuilder:rbac:groups=metal3.io,resources=hostupdatepolicies,verbs=get;list;watch
+//+kubebuilder:rbac:groups=metal3.io,resources=hostupdatepolicies,verbs=get;list;watch;update
 
 // Reconcile handles changes to BareMetalHost resources.
 func (r *BareMetalHostReconciler) Reconcile(ctx context.Context, request ctrl.Request) (result ctrl.Result, err error) {
-Original file line number
+Diff line change
@@ Expand Up / @@ -74,4 +74,5 @@ rules: @@
       verbs:
       - get
       - list
+      - update
       - watch