Skip to content
Permalink
Browse files
Fix chart rendering when annotations specified
Commit 70b2777 caused user-provided
annotations to break the `checksum` StatefulSet annotations. The
`{{ with .Values.annotations }}` instruction changes the scope
under which the checksum is calculated when user annotations are present.

This removes the `{{ with .Values.annotations }}` and instead just
writes out the user-provided annotations directly.

I've also added a user-provided annotation to the `values.yaml` used
in the smoke test to verify the fix.

Fixes #37
  • Loading branch information
willholley authored and kocolosk committed Jan 7, 2022
1 parent b995942 commit ffd44b645283410255a012bae0f7d822dec5b5ca
Showing 2 changed files with 4 additions and 2 deletions.
@@ -1,3 +1,5 @@
couchdbConfig:
couchdb:
uuid: "decafbaddecafbaddecafbaddecafbad"
annotations:
foo: bar
@@ -18,10 +18,10 @@ spec:
metadata:
labels:
{{ include "couchdb.ss.selector" . | indent 8 }}
{{- with .Values.annotations }}
annotations:
checksum/config: {{ include (print $.Template.BasePath "/configmap.yaml") . | sha256sum }}
{{ toYaml . | indent 8 }}
{{- if .Values.annotations }}
{{ toYaml .Values.annotations | indent 8 }}
{{- end }}
spec:
{{- if .Values.schedulerName }}

0 comments on commit ffd44b6

Please sign in to comment.