Skip to content

Commit

Permalink
chore: Change default values for Bitnami Postgres helm chart (#71)
Browse files Browse the repository at this point in the history
* Change default values for Bitnami Postgres helm chart
* Use password defined in `postgresql.auth.password`
  • Loading branch information
sturman authored Mar 7, 2023
1 parent 3e95791 commit 9f50212
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 9 deletions.
2 changes: 1 addition & 1 deletion charts/unleash/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,4 @@ sources:
- https://github.com/Unleash/unleash
- https://github.com/Unleash/helm-charts
type: application
version: 2.7.6
version: 2.7.7
2 changes: 1 addition & 1 deletion charts/unleash/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ spec:
valueFrom:
secretKeyRef:
name: {{ if .Values.dbConfig.useExistingSecret.name }}{{ .Values.dbConfig.useExistingSecret.name }}{{ else }}{{ .Values.postgresql.fullnameOverride }}{{ end }}
key: {{ if .Values.dbConfig.useExistingSecret.key }}{{ .Values.dbConfig.useExistingSecret.key }}{{ else }}postgres-password{{ end }}
key: {{ if .Values.dbConfig.useExistingSecret.key }}{{ .Values.dbConfig.useExistingSecret.key }}{{ else }}password{{ end }}
{{- end }}
- name: DATABASE
value: "{{ .Values.dbConfig.database }}"
Expand Down
15 changes: 8 additions & 7 deletions charts/unleash/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -109,17 +109,17 @@ configMaps: {}
containerPort: 4242

dbConfig:
# if postgres dependency chart is used, this needs to be the same value as postgresql.postgresqlDatabase
# if postgres dependency chart is used, this needs to be the same value as postgresql.auth.database
database: unleash
host: postgres
# if postgres dependency chart is used, this needs to be the same value as postgresql.postgresqlPassword
# if postgres dependency chart is used, this needs to be the same value as postgresql.auth.password
# this should be change to something more secure or use an existing secret
pass: unleash
port: 5432
useExistingSecret:
name: ""
key: ""
# if postgres dependency chart is used, this needs to be the same value as postgresql.postgresqlUsername
# if postgres dependency chart is used, this needs to be the same value as postgresql.auth.username
user: unleash
ssl: false

Expand Down Expand Up @@ -231,10 +231,11 @@ vpa:
postgresql:
enabled: true
fullnameOverride: "unleash-postgresql"
postgresqlDatabase: unleash
postgresqlUsername: unleash
# this should be change to something more secure or use an existing secret
postgresqlPassword: unleash
auth:
username: unleash
# this should be change to something more secure or use an existing secret
password: unleash
database: unleash

securityContext: {}
# Autoscaling
Expand Down

0 comments on commit 9f50212

Please sign in to comment.