make livenessprobe consistent across manifests; make helm chart configurable via values.yaml

charts/descheduler/templates/cronjob.yaml

@@ -68,6 +68,8 @@ spec:
                 - {{ $value | quote }}
                 {{- end }}
                 {{- end }}
+              livenessProbe:
+                {{- toYaml .Values.livenessProbe | nindent 16 }}
               resources:
                 {{- toYaml .Values.resources | nindent 16 }}
               securityContext:

charts/descheduler/templates/deployment.yaml

@@ -48,13 +48,7 @@ spec:
             - containerPort: 10258
               protocol: TCP
           livenessProbe:
-            failureThreshold: 3
-            httpGet:
-              path: /healthz
-              port: 10258
-              scheme: HTTPS
-            initialDelaySeconds: 3
-            periodSeconds: 3
+            {{- toYaml .Values.livenessProbe | nindent 12 }}
           resources:
             {{- toYaml .Values.resources | nindent 12 }}
           securityContext:

charts/descheduler/values.yaml

@@ -94,3 +94,12 @@ serviceAccount:
   # The name of the ServiceAccount to use.
   # If not set and create is true, a name is generated using the fullname template
   name:
+
+livenessProbe:
+  failureThreshold: 3
+  httpGet:
+    path: /healthz
+    port: 10258
+    scheme: HTTPS
+  initialDelaySeconds: 3
+  periodSeconds: 10

kubernetes/deployment/deployment.yaml

@@ -40,7 +40,7 @@ spec:
               port: 10258
               scheme: HTTPS
             initialDelaySeconds: 3
-            periodSeconds: 3
+            periodSeconds: 10
           resources:
             requests:
               cpu: 500m

kubernetes/job/job.yaml

@@ -36,7 +36,7 @@ spec:
               port: 10258
               scheme: HTTPS
             initialDelaySeconds: 3
-            periodSeconds: 3
+            periodSeconds: 10
           securityContext:
             allowPrivilegeEscalation: false
             capabilities: