Skip to content

Commit

Permalink
Fix maintenance Job resource requests (#610)
Browse files Browse the repository at this point in the history
- As written, the chart tries to access
  .configuration.repositoryMaintenanceJob.requests.cpu.cpu and
...memory.memory, which leads to an error if you actually try to set
these values.

Signed-off-by: Matt Yaraskavitch <62650344+yaraskm@users.noreply.github.com>
  • Loading branch information
yaraskm authored Jul 30, 2024
1 parent 721abcb commit 88ecfa4
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions charts/velero/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,11 +3,11 @@ appVersion: 1.14.0
kubeVersion: ">=1.16.0-0"
description: A Helm chart for velero
name: velero
version: 7.1.3
version: 7.1.4
home: https://github.com/vmware-tanzu/velero
icon: https://cdn-images-1.medium.com/max/1600/1*-9mb3AKnKdcL_QD3CMnthQ.png
sources:
- https://github.com/vmware-tanzu/velero
- https://github.com/vmware-tanzu/velero
maintainers:
- name: jenting
email: hsiaoairplane@gmail.com
Expand Down
4 changes: 2 additions & 2 deletions charts/velero/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -169,10 +169,10 @@ spec:
{{- with .repositoryMaintenanceJob }}
{{- with .requests }}
{{- with .cpu }}
- --maintenance-job-cpu-request={{ .cpu }}
- --maintenance-job-cpu-request={{ . }}
{{- end }}
{{- with .memory }}
- --maintenance-job-mem-request={{ .memory }}
- --maintenance-job-mem-request={{ . }}
{{- end }}
{{- end }}
{{- with .limits }}
Expand Down

0 comments on commit 88ecfa4

Please sign in to comment.