Unleash / helm-charts

Contains helm-charts for Unleash
Apache License 2.0
41 stars 56 forks source link

Error: couldn't find key `postgresql-password` in Secret `unleash-postgresql` #69

Closed sturman closed 1 year ago

sturman commented 1 year ago

Describe the bug

Wrong Secret key is used to reference POSTGRESQL_PASSWORD in Unleash Deployment when using default helm chart values

Steps to reproduce the bug

  1. Add Unleash helm repo
    helm repo add unleash https://docs.getunleash.io/helm-charts
  2. Template Unleash helm chart
    helm template unleash/unleash
  3. Find Deployment of Unleash
  4. Check environment variable POSTGRESQL_PASSWORD reference

Actual: Deployment manifest:

...
spec:
  containers:
    - name: unleash
      env:
        - name: POSTGRESQL_PASSWORD
          valueFrom:
            secretKeyRef:
              name: unleash-postgresql
              key: postgresql-password
...

Secret manifest:

apiVersion: v1
kind: Secret
metadata:
  name: unleash-postgresql
  namespace: "dev"
  labels:
    app.kubernetes.io/name: postgresql
    helm.sh/chart: postgresql-12.1.6
    app.kubernetes.io/instance: release-name
    app.kubernetes.io/managed-by: Helm
type: Opaque
data:
  postgres-password: "Y2lpTlNIbm9xTA=="
  # We don't auto-generate LDAP password when it's not provided as we do for other passwords

The problem is that the Secret does not contain postgresql-password. It has postgres-password key instead.

Expected behavior

No response

Logs, error output, etc.

No response

Screenshots

No response

Additional context

No response

Unleash version

No response

Subscription type

Open source

Hosting type

Self-hosted

SDK information (language and version)

No response

thomasheartman commented 1 year ago

@sturman Thanks for opening the issue, for the clear report, and for the PR! We'll process it as soon as we can!