[bitnami/postgresql] Configurable DATA_SOURCE_USER and similar from values of postgres exporter

[snosratiershad]

Name and Version

bitnami/postgresql main

What is the problem this feature will solve?

The exporter user could be any user with access to postgres database and granted pg_monitor. It's not uncommon to separate it from admin user. This feature will solve the problem of changing user/pass etc in a clean way from values without modifications to hard-coded value in statefulset template directly.

What is the feature you are proposing to solve the problem?

A variable in postgres exporter values, mapped to DATA_SOURCE_* env variables.

What alternatives have you considered?

Maybe also add mapping with configmap or secrets

[snosratiershad]

Please let me know if this approach is welcome and I'll send a PR for it.

[javsalgar]

Hi!

Thank you so much for the feature request! Indeed, it would be a welcome feature, so feel free to submit a PR :D

[snosratiershad]

I went into some progress.
First: I decided to place the metricsUser and metricsPassword inside auth instead of metrics. which is different from the way done on ldap password. one of the reason was that postgres exporter also accepts password files and I wanted to using password files and the key name configuration to modify it's behavior so it's better to keep them in same place.

Second: I decided to not autogenerate the user if something is provided. just checking if it's not empty or "postgres" and configuring the secret and use it.

@javsalgar @carrodher
Do you think I'm in the righ path?

[javsalgar]

Hi,

And what about metrics.auth?

[snosratiershad]

@javsalgar this is what was in my mind: #36465

do you think I should change it to metrics.auth?