Allow overriding the image tag completely

[joshdcu]

Please see issue #82 for details

[allanger]

I wanted to get rid of the v prefix for several years now, but never had enough will to actually do it, because I'm not sure how much of a breaking change it would be.

Regarding the pr, you need to bump the chart version, otherwise linter won't pass. But the question is, which version it should be. It seems like a breaking change to me, so maybe you need to bump the major one.

Actually, maybe I would remove the v prefix completely and as a temporary workaround would add it to the Chart.yaml to the app version, and with the next db-operator release, I would remove it completely from the tags.

And also, please add a note here: https://github.com/db-operator/charts/blob/main/charts/db-operator/templates/NOTES.txt

Just remove the content of "Breaking changes and migration paths:" and instead mention that if the tag was used in the values before, it should be now prefixed with v explicitly.

[joshdcu]

Thanks for your review! I'll make the changes you've mentioned soon 👍

[joshdcu]

you need to bump the chart version, otherwise linter won't pass. But the question is, which version it should be. It seems like a breaking change to me, so maybe you need to bump the major one.

Bumped to v2.0.0

maybe I would remove the v prefix completely and as a temporary workaround would add it to the Chart.yaml to the app version

Done!

please add a note here: https://github.com/db-operator/charts/blob/main/charts/db-operator/templates/NOTES.txt

Added a note there and also updated the README.md

[allanger]

It seems like the tests are broken, but it shouldn't have anything to do with this PR, I'll have a look later

[allanger]

Thanks!