Merge pull request #1573 from icloudnote/charts

Fixed the issue when successfulJobsHistoryLimit and failedJobsHistoryLimit variables are 0.
kubernetes-sigs · Dec 14, 2024 · 5c3a3bd · 5c3a3bd
2 parents 46fa370 + bc63236
commit 5c3a3bd
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/charts/descheduler/templates/cronjob.yaml b/charts/descheduler/templates/cronjob.yaml
@@ -15,10 +15,10 @@ spec:
   {{- if .Values.startingDeadlineSeconds }}
   startingDeadlineSeconds: {{ .Values.startingDeadlineSeconds }}
   {{- end }}
-  {{- if .Values.successfulJobsHistoryLimit }}
+  {{- if ne .Values.successfulJobsHistoryLimit nil }}
   successfulJobsHistoryLimit: {{ .Values.successfulJobsHistoryLimit }}
   {{- end }}
-  {{- if .Values.failedJobsHistoryLimit }}
+  {{- if ne .Values.failedJobsHistoryLimit nil }}
   failedJobsHistoryLimit: {{ .Values.failedJobsHistoryLimit }}
   {{- end }}
   {{- if .Values.timeZone }}