search

movetokube / postgres-operator

Postgres operator for Kubernetes
MIT License
167 stars 59 forks source link

Helm Chart should come with .image.tag set to an empty string in values.yaml #131

Closed aszeszo closed 9 months ago

aszeszo commented 1 year ago

Thank you for all the work on the operator. I find it very useful!

There was a small change that was made as part of the following commit: https://github.com/movetokube/postgres-operator/commit/d5f48aa67c52de1c52469f85be60bf0c1483b525

tag: "" -> tag: "latest" in charts/ext-postgres-operator/values.yaml

I suspect it was not intentional. It causes Helm-based deployments use the latest tag, instead of the appVersion one as specified in the Chart.yaml file. This may result in unpredictable behaviour if .image.tag value is not explicitly set in the local values file, and the latest registry tag is updated to point at incompatible image.