[DB] Fix builtin DB init script. Document the DB existingSecret params

jeanpommier commented 10 months ago

Init Script:

Fix #48: name of init script should match the one provided in values.yaml
In case of Builtin DBs: Init script was giving ownership of all DBs to georchestra user, no using the config params for geodata and datafeeder DBs. Which were nevertheless used by the geodata and datafeeder connection secrets. So for instance datafeeder was trying to connect with datafeeder user, but it had not been created. This way, the init script actually creates the users and gives them ownership on their respective DB

Documentation: clarify the usage of the existingSecret params

jeanpommier commented 9 months ago

@edevosc2c @jeanmi151 would you have a minute to review this one please ?

jeanpommier commented 9 months ago

did you test it with other password than default one ?

It was 2 weeks ago, so I don't remember (but suppose so). Checked it right now, seems fine If it's OK for you, I'll merge this one. I'd like to avoid having to refresh it it the chart changes in between

georchestra / helm-georchestra

[DB] Fix builtin DB init script. Document the DB existingSecret params #51