Skip to content

Commit

Permalink
fix(template/helm): Add checksum/config annotation (#480)
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
NickLarsenNZ authored Feb 3, 2025
1 parent 458ab02 commit d64a1bb
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,8 @@ spec:
metadata:
annotations:
internal.stackable.tech/image: "{{ .Values.image.repository }}:{{ .Values.image.tag | default .Chart.AppVersion }}"
{{- with .Values.podAnnotations }}
checksum/config: {{ include (print $.Template.BasePath "/configmap.yaml") . | sha256sum }}
{{- with .Values.podAnnotations }}
{{- toYaml . | nindent 8 }}
{{- end }}
labels:
Expand Down

0 comments on commit d64a1bb

Please sign in to comment.