From d7ca2ab40ddb1a91480e6df320ff4ccf387aaf33 Mon Sep 17 00:00:00 2001
From: Nick Larsen <nick.larsen@stackable.tech>
Date: Wed, 25 Mar 2026 16:06:47 +0100
Subject: [PATCH 1/2] chore: Describe RBAC rules, remove unnecessary rules

---
 .../commons-operator/templates/roles.yaml     | 22 ++++++++++++++-----
 1 file changed, 17 insertions(+), 5 deletions(-)

diff --git a/deploy/helm/commons-operator/templates/roles.yaml b/deploy/helm/commons-operator/templates/roles.yaml
index 31d541f..a9ab425 100644
--- a/deploy/helm/commons-operator/templates/roles.yaml
+++ b/deploy/helm/commons-operator/templates/roles.yaml
@@ -6,33 +6,43 @@ metadata:
   labels:
   {{- include "operator.labels" . | nindent 4 }}
 rules:
+  # Watch pods to detect expiry annotations and evict them (pod restart controller).
+  # Watch configmaps and secrets (metadata only) to detect changes that should trigger a
+  # rolling restart of referencing StatefulSets (StatefulSet restart controller).
+  # list + watch are sufficient; no individual get calls are made (Controller and
+  # metadata_watcher use list + watch internally).
   - apiGroups:
       - ""
     resources:
       - pods
       - configmaps
       - secrets
-      - nodes
     verbs:
-      - get
       - list
       - watch
-  # For automatic cluster domain detection
+  # For automatic cluster domain detection: proxy to the kubelet configz endpoint on the
+  # operator's own node (name supplied via the downward API) to read the clusterDomain
+  # setting. Only a direct GET on the named node's proxy subresource is needed - no
+  # list or watch of nodes is required.
   - apiGroups:
       - ""
     resources:
       - nodes/proxy
     verbs:
       - get
+  # Watch StatefulSets labelled restarter.stackable.tech/enabled=true (list + watch) and
+  # patch their pod template annotations via Server-Side Apply to trigger rolling restarts
+  # when referenced ConfigMaps or Secrets change (patch). No get needed: SSA does not
+  # require a preceding get, and the reflector/watcher covers list + watch.
   - apiGroups:
       - apps
     resources:
       - statefulsets
     verbs:
-      - get
       - list
       - watch
-      - patch # We need to add a label to the StatefulSet
+      - patch
+  # Emit Kubernetes events from both the StatefulSet and Pod restart controllers.
   - apiGroups:
       - events.k8s.io
     resources:
@@ -40,6 +50,8 @@ rules:
     verbs:
       - create
       - patch
+  # Evict pods whose restarter.stackable.tech/expires-at.* annotation timestamp has been
+  # reached (pod restart controller). Eviction is a create on the pods/eviction subresource.
   - apiGroups:
       - ""
     resources:

From bd73246dbcb931f775215647f401bd450e99d5ec Mon Sep 17 00:00:00 2001
From: Nick Larsen <nick.larsen@stackable.tech>
Date: Wed, 25 Mar 2026 16:08:47 +0100
Subject: [PATCH 2/2] chore: Update changelog

---
 CHANGELOG.md | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 4f5aee6..8777b3e 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -4,6 +4,12 @@ All notable changes to this project will be documented in this file.
 
 ## [Unreleased]
 
+### Changed
+
+- Helm deployed RBAC permissions documented, with unnecessary permissions removed ([#412]).
+
+[#412]: https://github.com/stackabletech/commons-operator/pull/412
+
 ## [26.3.0] - 2026-03-16
 
 ## [26.3.0-rc1] - 2026-03-16