paralus / helm-charts

Helm charts for Paralus
https://www.paralus.io/
12 stars 15 forks source link

Postgresql password not being properly output in the DSN #44

Closed mcfearsome closed 2 years ago

mcfearsome commented 2 years ago

Expected vs actual behavior

I expect the DSN in secrets to include the configured password.

<no value> is output where the password should be.

Steps to reproduce the bug

  1. disable the embedded postgresql install .Values.deploy.postgresql.enable=false
  2. configure the required postgresql variables for building the DSN
  3. run helm template . and inspect the paralus-db secret.

Are you using the latest version of the project?

  chart:
    spec:
      chart: ztka
      version: "~> 0.1.0"
      sourceRef:
        kind: HelmRepository
        name: paralus

-

Notes

Will open a PR fixing this issue but it also contains other changes to allow the explicit setting of the postgresql DSN

meain commented 2 years ago

Woops, great catch. Looks like this fell through the cracks.