Merge pull request #827 from shkrid/leader-election-for-controller

Allow leader election for scheduler-plugins-controller in Helm chart
kubernetes-sigs · Nov 12, 2024 · 5b7116b · 5b7116b
2 parents e42fe3d + 8438d1e
commit 5b7116b
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 0 deletions.
diff --git a/manifests/install/charts/as-a-second-scheduler/templates/deployment.yaml b/manifests/install/charts/as-a-second-scheduler/templates/deployment.yaml
@@ -21,6 +21,10 @@ spec:
       serviceAccountName: {{ .Values.controller.name }}
       containers:
       - name: scheduler-plugins-controller
+        args:
+        {{- if .Values.controller.leaderElect }}
+        - --enableLeaderElection
+        {{- end }}
         image: {{ .Values.controller.image }}
         imagePullPolicy: IfNotPresent
         {{- with .Values.controller.resources }}

diff --git a/manifests/install/charts/as-a-second-scheduler/templates/rbac.yaml b/manifests/install/charts/as-a-second-scheduler/templates/rbac.yaml
@@ -109,6 +109,9 @@ rules:
 - apiGroups: [""]
   resources: ["events"]
   verbs: ["create", "patch", "update"]
+- apiGroups: ["coordination.k8s.io"]
+  resources: ["leases"]
+  verbs: ["create", "get", "update"]
 - apiGroups: [""]
   resources: ["nodes"]
   verbs: ["get", "list", "watch", "patch"]

diff --git a/manifests/install/charts/as-a-second-scheduler/values.yaml b/manifests/install/charts/as-a-second-scheduler/values.yaml
@@ -19,6 +19,7 @@ controller:
   name: scheduler-plugins-controller
   image: registry.k8s.io/scheduler-plugins/controller:v0.30.6
   replicaCount: 1
+  leaderElect: false
   priorityClassName: ""
   resources: {}
   nodeSelector: {}