Skip to content

Commit d64a1bb

Browse files
authored
fix(template/helm): Add checksum/config annotation (#480)
Add the dynamic `checksum/config` annotation back in. Previously, this annotation only appeared when `.Values.podAnnotations` was set. It was only discovered broken by adding podAnnotations and seeing that the `include` context was invalid, leading to this error: ``` Error: template: trino-operator/templates/deployment.yaml:20:28: executing "trino-operator/templates/deployment.yaml" at <include (print $.Template.BasePath "/configmap.yaml") .>: error calling include: template: trino-operator/templates/configmap.yaml:4:10: executing "trino-operator/templates/configmap.yaml" at <.Files.Glob>: nil pointer evaluating interface {}.Glob ``` Note, this was always broken since inception in #7.
1 parent 458ab02 commit d64a1bb

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

template/deploy/helm/[[operator]]/templates/deployment.yaml.j2

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,8 +17,8 @@ spec:
1717
metadata:
1818
annotations:
1919
internal.stackable.tech/image: "{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"
20-
{{- with .Values.podAnnotations }}
2120
checksum/config: {{ include (print $.Template.BasePath "/configmap.yaml") . | sha256sum }}
21+
{{- with .Values.podAnnotations }}
2222
{{- toYaml . | nindent 8 }}
2323
{{- end }}
2424
labels:

0 commit comments

Comments
 (0)