influxdata / helm-charts

Official Helm Chart Repository for InfluxData Applications
MIT License
233 stars 330 forks source link

Add support for podAnnotations helm rollme and/or checksum/config #658

Open cristian98149 opened 4 months ago

cristian98149 commented 4 months ago

Hi,

as in helm official docs (https://helm.sh/docs/howto/charts_tips_and_tricks/#automatically-roll-deployments), rollme and checksum/config annotations can be used in order to restart pod after a helm upgrade or configmap changes.

Currently, it is not possible to specify via values.yaml "rollme: {{ randAlphaNum 5 | quote }}" and "checksum/config: {{ include (print $.Template.BasePath "/configmap.yaml") . | sha256sum }}" while they contain templating function that cannot be specified in values file.

I would suggest to add "rollme: {{ randAlphaNum 5 | quote }}" in spec.template.metadata.annotations field in statefulset.yaml influxdb2 chart template and use a boolean value in values.yaml (rollme for example) for controlling this annotation.

This could be very useful.