Skip to content

Commit

Permalink
Merge pull request #444 from KohlsTechnology/issue-432-cronjob
Browse files Browse the repository at this point in the history
Update Helm Chart To Allow Setting startingDeadlineSeconds
  • Loading branch information
k8s-ci-robot authored Nov 24, 2020
2 parents 5f1e9a9 + 03852d0 commit 7bcd562
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 0 deletions.
1 change: 1 addition & 0 deletions charts/descheduler/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,7 @@ The following table lists the configurable parameters of the _descheduler_ chart
| `nameOverride` | String to partially override `descheduler.fullname` template (will prepend the release name) | `""` |
| `fullnameOverride` | String to fully override `descheduler.fullname` template | `""` |
| `schedule` | The cron schedule to run the _descheduler_ job on | `"*/2 * * * *"` |
| `startingDeadlineSeconds` | If set, configure `startingDeadlineSeconds` for the _descheduler_ job | `nil` |
| `cmdOptions` | The options to pass to the _descheduler_ command | _see values.yaml_ |
| `deschedulerPolicy.strategies` | The _descheduler_ strategies to apply | _see values.yaml_ |
| `priorityClassName` | The name of the priority class to add to pods | `system-cluster-critical` |
Expand Down
3 changes: 3 additions & 0 deletions charts/descheduler/templates/cronjob.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,9 @@ metadata:
spec:
schedule: {{ .Values.schedule | quote }}
concurrencyPolicy: "Forbid"
{{- if .Values.startingDeadlineSeconds }}
startingDeadlineSeconds: {{ .Values.startingDeadlineSeconds }}
{{- end }}
jobTemplate:
spec:
template:
Expand Down
1 change: 1 addition & 0 deletions charts/descheduler/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@ nameOverride: ""
fullnameOverride: ""

schedule: "*/2 * * * *"
#startingDeadlineSeconds: 200

cmdOptions:
v: 3
Expand Down

0 comments on commit 7bcd562

Please sign in to comment.