Skip to content

Commit cbce408

Browse files
committed
Avoid proxy rollouts on chart metadata changes
Summary: Hash only the rendered HAProxy configuration for the pod-template checksum instead of the entire ConfigMap manifest. This prevents chart label and other metadata-only changes from triggering a Deployment rollout. Test Plan: - `helm lint charts/whatsapp-proxy-chart --values /data/users/bsuh/configerator/raw_configs/cloud/helm/values/wa_unblock/whatsapp-proxy-chart/common_values.yaml --values /data/users/bsuh/configerator/raw_configs/cloud/helm/values/wa_unblock/whatsapp-proxy-chart/chat_values.yaml` - Rendered the chart with `helm template` and verified the checksum is a valid SHA-256 value. - Changed the chart version to `9.9.9` in a temporary chart copy and verified the rendered HAProxy checksum remained unchanged. Reviewers: Subscribers: Tasks: Tags:
1 parent 8b06a45 commit cbce408

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

charts/whatsapp-proxy-chart/templates/deployment.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ spec:
1818
template:
1919
metadata:
2020
annotations:
21-
checksum/haproxy-config: {{ include (print $.Template.BasePath "/configmap.yaml") . | sha256sum }}
21+
checksum/haproxy-config: {{ tpl .Values.haproxyConfig . | sha256sum }}
2222
{{- with .Values.podAnnotations }}
2323
{{- toYaml . | nindent 8 }}
2424
{{- end }}

0 commit comments

Comments
 (0)