Skip to content

Avoid proxy rollouts on chart metadata changes#538

Merged
bsuh merged 1 commit into
mainfrom
pr538
Jul 22, 2026
Merged

Avoid proxy rollouts on chart metadata changes#538
bsuh merged 1 commit into
mainfrom
pr538

Conversation

@bsuh

@bsuh bsuh commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

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.

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 22, 2026
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:
@bsuh
bsuh merged commit 9b68605 into main Jul 22, 2026
5 checks passed
@bsuh
bsuh deleted the pr538 branch July 22, 2026 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants