Environment variables names are no correct about helm-charts

[stevehipwell]

@oisis where are you sourcing this information from? The official docs show the current variables as correct and the installation guide shows the variables that you've quoted as deprecated.

[oisis]

I was trying to setup SonarQube with Postgres on K8s with your chart and i couldn't get it working. After some small investigation all was set up properly but had to change env variables names. Please check this:
https://github.com/SonarSource/docker-sonarqube/blob/master/7/community/Dockerfile#L7-L11
https://github.com/SonarSource/docker-sonarqube/blob/master/8/community/run.sh#L49-L51
I believe, you have to use the same ENV VAR names as is in SonarQube docker images.

[stevehipwell]

@oisis the latest chart is for v9 and doesn't support v7, if it works so be it but I doubt it would, or v8, which should work fine as there haven't been any chart changes that would stop this. If you want to use v7 or v8 in a supported way please use an older chart version.

To clarify the issue you reported (I'm assuming that you're on v8 or v9) the function you've highlighted set_prop_from_deprecated_env_var specifically calls out that the env variables you so confidently told me I should swap in are in fact deprecated, I gave you the link to the official documentation and a deprecation notice above.

These charts are provided as is and with the usual advice of not running DBs in K8s; I personally use RDS instances and can confirm that the env variables in the chart work correctly. I wouldn't go as far as to say that the embedded PSQL chart definitely work, although it did last time I tested it. I'm happy to look at the values you're providing to see if there is anything obvious but besides that you could use a different source for PSQL, such as a separate chart, and pass in the connection values.

[oisis]

Yes, you are right, i was trying to setup SQ 7 with latest chart version. Will try next to setup later SQ 8 and 9(upgrade process from 7) with this chart. Anyway thanks for your help. If chart is working ok with SQ9 i think you can close this ticket.

[stevehipwell]

@oisis I remembered wrong, I've never supported v7 in my chart but there is a v7 chart in the legacy repo.

In general a Helm chart should be compatible with a different patch version to the one it's released with; it might be compatible with a different minor version; but it's unlikely to be compatible with a different major version. This rule is made even more likely with something complex, like SonarQube, or something not necessarily designed for Kubernetes or even containers, like SonarQube.