Skip to content

Commit b418dd1

Browse files
committed
[SPARK-57917] Fix workload RoleBinding name in operator namespace
### What changes were proposed in this pull request? This PR fixes the workload `RoleBinding` name in `workload-rbac.yaml` to use `$workloadResources.roleBinding.name` instead of `$workloadResources.serviceAccount.name` when `workloadResources.namespaces.data` is empty, consistent with the per-namespace branch. ### Why are the changes needed? The `RoleBinding` was named `spark` (the service account name) instead of the intended `spark-workload-rolebinding`. ### Does this PR introduce _any_ user-facing change? Yes. The `RoleBinding` in the operator namespace is now named `spark-workload-rolebinding` instead of `spark`. ### How was this patch tested? Pass the CIs. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: Claude Fable 5 Closes #739 from dongjoon-hyun/dongjoon/nervous-leavitt-747dd2. Authored-by: Dongjoon Hyun <dongjoon@apache.org> Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
1 parent 4077c8d commit b418dd1

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,7 @@ metadata:
144144
apiVersion: rbac.authorization.k8s.io/v1
145145
kind: RoleBinding
146146
metadata:
147-
name: {{ $workloadResources.serviceAccount.name }}
147+
name: {{ $workloadResources.roleBinding.name }}
148148
namespace: {{ $systemNs }}
149149
{{- template "spark-operator.workloadLabelsAnnotations" $ }}
150150
{{- template "spark-operator.workloadRoleRef" $ }}

0 commit comments

Comments
 (0)