Skip to content

Commit 76761c2

Browse files
committed
[SPARK-59537] Honor operatorRbac.configManagement.create in Helm chart
### What changes were proposed in this pull request? This PR aims to honor `operatorRbac.configManagement.create` in the Helm chart. ### Why are the changes needed? Previously, this value was ignored, so setting it to `false` did not prevent the config-monitor `Role` and `RoleBinding` from being created. ### Does this PR introduce _any_ user-facing change? Yes, `operatorRbac.configManagement.create=false` now takes effect. The default is `true`, so the default behavior is unchanged. ### How was this patch tested? Pass the CIs and manual `helm template` checks. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: Claude Opus 5 Closes #838 from dongjoon-hyun/SPARK-59537. Authored-by: Dongjoon Hyun <dongjoon@apache.org> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
1 parent 4ca2f71 commit 76761c2

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

‎build-tools/helm/spark-kubernetes-operator/templates/operator-rbac.yaml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -235,7 +235,7 @@ metadata:
235235
{{- template "spark-operator.operatorClusterRbacRules" $ }}
236236
---
237237
{{- end }}
238-
{{- if and (eq (include "spark-operator.dynamicConfig.enabled" .) "true") (eq .Values.operatorConfiguration.dynamicConfig.source "configMap") }}
238+
{{- if and .Values.operatorRbac.configManagement.create (eq (include "spark-operator.dynamicConfig.enabled" .) "true") (eq .Values.operatorConfiguration.dynamicConfig.source "configMap") }}
239239
apiVersion: rbac.authorization.k8s.io/v1
240240
kind: Role
241241
metadata:

0 commit comments

Comments
 (0)