github-actions[bot] opened a new pull request, #73048: URL: https://github.com/apache/airflow/pull/73048
* Fix StatsD not restarting when overrideMappings or cache.ttl changes The checksum annotation that rolls the StatsD pod was only rendered when extraMappings or podAnnotations was set, so changing overrideMappings or cache.ttl updated the ConfigMap without restarting the pod and the exporter silently kept the old mappings. The checksum is also narrowed to the ConfigMap data so that chart version bumps and other metadata-only changes no longer restart StatsD. * Hash the whole StatsD ConfigMap document for the rollout checksum Narrowing the checksum to the ConfigMap's .data made StatsD the only component in the chart whose pods do not roll on a chart version bump. Review preferred keeping the same full-document hash every other component uses: a restart of the stateless StatsD exporter on upgrade is cheap, and a change we would want to roll out for but did not anticipate is not. * Keep the podAnnotations block in its original form The if-to-with rewrite was a cosmetic change unrelated to the rollout fix and was flagged as unnecessary in review. --------- (cherry picked from commit 81b93db4a2f6da3d4e56a337f0f0b84fb2f1b029) Co-authored-by: Y-C <[email protected]> Co-authored-by: Eason09053360 <[email protected]> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
