Skip to content

Commit 473d1dc

Browse files
committed
[G2P-3424] Several Helm templating bugs fixed. Chart installs without errors on Cluster.
1 parent 116828f commit 473d1dc

File tree

6 files changed

+25
-25
lines changed

6 files changed

+25
-25
lines changed

charts/openg2p-pbms-bg-task-api/templates/virtualservice.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,14 +10,14 @@ metadata:
1010
{{- end }}
1111
spec:
1212
hosts:
13-
- {{ default .Values.global.g2pPbmsBgTaskHostname .Values.istio.virtualservice.host | quote }}
13+
- {{ default ( include "common.tplvalues.render" (dict "value" .Values.global.g2pPbmsBgTaskHostname "context" $ )) .Values.istio.virtualservice.host | quote }}
1414
gateways:
1515
- {{ default (include "common.names.fullname" .) .Values.istio.virtualservice.gateway }}
1616
http:
1717
- headers:
1818
request:
1919
set:
20-
x-forwarded-host: {{ default .Values.global.g2pPbmsBgTaskHostname .Values.istio.virtualservice.host | quote }}
20+
x-forwarded-host: {{ default (include "common.tplvalues.render" (dict "value" .Values.global.g2pPbmsBgTaskHostname "context" $)) .Values.global.g2pPbmsBgTaskHostname .Values.istio.virtualservice.host | quote }}
2121
x-forwarded-proto: https
2222
match:
2323
- uri:

charts/openg2p-pbms-bg-task-api/values.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,9 @@ global:
1010
registryDBUserPasswordKey: 'registry_db_user'
1111
postgresqlHost: openg2p-commons-postgresql
1212
pbmsBgTaskDB: '{{ .Release.Name }}_bgtask_db'
13-
pbmsBgTaskDBUser: '{{ .Release.Name}}_bgtask_db_user'
13+
pbmsBgTaskDBUser: '{{ .Release.Name }}_bgtask_db_user'
1414
pbmsBgTaskDBSecret: '{{ .Release.Name }}_bgtask'
15-
pbmsBgTaskDBUserPasswordKey: '{{ .Release.Name}}_bgtask_db_user'
15+
pbmsBgTaskDBUserPasswordKey: '{{ .Release.Name }}_bgtask_db_user'
1616

1717
## Pod extra labels
1818
## ref: https://kubernetes.io/docs/concepts/overview/working-with-objects/labels/

charts/openg2p-pbms-bg-task-celery-beat-producers/values.yaml

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -10,14 +10,14 @@ global:
1010
registryDBUserPasswordKey: 'registry_db_user'
1111

1212
pbmsDB: '{{ .Release.Name }}_db'
13-
pbmsDBUser: '{{ .Release.Name}}_db_user'
13+
pbmsDBUser: '{{ .Release.Name }}_db_user'
1414
pbmsDBSecret: '{{ .Release.Name }}'
15-
pbmsDBUserPasswordKey: '{{ .Release.Name}}_db_user'
15+
pbmsDBUserPasswordKey: '{{ .Release.Name }}_db_user'
1616

1717
pbmsBgTaskDB: '{{ .Release.Name }}_bgtask_db'
18-
pbmsBgTaskDBUser: '{{ .Release.Name}}_bgtask_db_user'
18+
pbmsBgTaskDBUser: '{{ .Release.Name }}_bgtask_db_user'
1919
pbmsBgTaskDBSecret: '{{ .Release.Name }}_bgtask'
20-
pbmsBgTaskDBUserPasswordKey: '{{ .Release.Name}}_bgtask_db_user'
20+
pbmsBgTaskDBUserPasswordKey: '{{ .Release.Name }}_bgtask_db_user'
2121

2222
# Default name of Redis that is installed with the parent openg2p-pbms chart
2323
redisInstallationName: '{{ .Release.Name }}-redis-master'
@@ -201,8 +201,7 @@ envVarsFrom:
201201
BG_TASK_CELERY_BEAT_DB_PASSWORD_PBMS :
202202
secretKeyRef:
203203
name: '{{ tpl .Values.global.pbmsDBSecret $ }}'
204-
#key: '{{ tpl .Values.global.pbmsDBuserPasswordKey $ }}'
205-
key: '{{ .Values.global.pbmsDBuserPasswordKey }}'
204+
key: '{{ tpl .Values.global.pbmsDBUserPasswordKey $ }}'
206205

207206
## ref: http://kubernetes.io/docs/user-guide/compute-resources/
208207
##

charts/openg2p-pbms-bg-task-celery-workers/values.yaml

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -10,14 +10,14 @@ global:
1010
registryDBUserPasswordKey: 'registry_db_user'
1111

1212
pbmsDB: '{{ .Release.Name }}_db'
13-
pbmsDBUser: '{{ .Release.Name}}_db_user'
13+
pbmsDBUser: '{{ .Release.Name }}_db_user'
1414
pbmsDBSecret: '{{ .Release.Name }}'
15-
pbmsDBUserPasswordKey: '{{ .Release.Name}}_db_user'
15+
pbmsDBUserPasswordKey: '{{ .Release.Name }}_db_user'
1616

1717
pbmsBgTaskDB: '{{ .Release.Name }}_bgtask_db'
18-
pbmsBgTaskDBUser: '{{ .Release.Name}}_bgtask_db_user'
19-
pbmsBgTaskDBSecret: '{{ .Release.Name }}_bgtask'
20-
pbmsBgTaskDBUserPasswordKey: '{{ .Release.Name}}_bgtask_db_user'
18+
pbmsBgTaskDBUser: '{{ .Release.Name }}_bgtask_db_user'
19+
pbmsBgTaskDBSecret: '{{ .Release.Name }}-bgtask'
20+
pbmsBgTaskDBUserPasswordKey: '{{ .Release.Name }}_bgtask_db_user'
2121

2222
# Default name of Redis that is installed with the parent openg2p-pbms chart
2323
redisInstallationName: '{{ .Release.Name }}-redis-master'
@@ -220,8 +220,7 @@ envVarsFrom:
220220
BG_TASK_CELERY_WORKERS_DB_PASSWORD_PBMS:
221221
secretKeyRef:
222222
name: '{{ tpl .Values.global.pbmsDBSecret $ }}'
223-
#key: '{{ tpl .Values.global.pbmsDBuserPasswordKey $ }}'
224-
key: '{{ .Values.global.pbmsDBuserPasswordKey }}'
223+
key: '{{ tpl .Values.global.pbmsDBUserPasswordKey $ }}'
225224

226225
## ref: http://kubernetes.io/docs/user-guide/compute-resources/
227226
##

charts/openg2p-pbms/.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
11
Chart.lock
22
*.tgz
33
charts/odoo
4+
charts/postgres-init

charts/openg2p-pbms/values.yaml

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -5,14 +5,15 @@ global:
55
postgresqlHost: openg2p-commons-postgresql
66

77
pbmsDB: '{{ .Release.Name }}_db'
8-
pbmsDBUser: '{{ .Release.Name}}_db_user'
8+
pbmsDBUser: '{{ .Release.Name }}_db_user'
99
pbmsDBSecret: '{{ .Release.Name }}'
10-
pbmsDBUserPasswordKey: '{{ .Release.Name}}_db_user'
10+
pbmsDBUserPasswordKey: '{{ .Release.Name }}_db_user'
1111

1212
pbmsBgTaskDB: '{{ .Release.Name }}_bgtask_db'
13-
pbmsBgTaskDBUser: '{{ .Release.Name}}_bgtask_db_user'
14-
pbmsBgTaskDBSecret: '{{ .Release.Name }}_bgtask'
15-
pbmsBgTaskDBUserPasswordKey: '{{ .Release.Name}}_bgtask_db_user'
13+
pbmsBgTaskDBUser: '{{ .Release.Name }}_bgtask_db_user'
14+
# Note underscore doesn't work here for Kubernetes secret, hence hyphen in the name
15+
pbmsBgTaskDBSecret: '{{ .Release.Name }}-bgtask'
16+
pbmsBgTaskDBUserPasswordKey: '{{ .Release.Name }}_bgtask_db_user'
1617

1718
# Defaults that should be overridden by questions.yaml
1819
registryDB: 'registry_db'
@@ -30,9 +31,9 @@ global:
3031
minioInstallationName: openg2p-commons-minio
3132

3233
# TODO: Add this to questions.yaml
33-
g2pPbmsBgTaskHostname: '{{ tpl .Release.Name $ }}-bg-task.devops.openg2p.org'
34+
g2pPbmsBgTaskHostname: '{{ .Release.Name }}-bg-task.devops.openg2p.org'
3435

35-
g2pPbmsBgTaskClientId: 'openg2p-pbms-bg-task-{{ .Release.Namespace }}'
36+
g2pPbmsBgTaskClientId: 'openg2p-pbms-bg-task-{{ tpl .Release.Namespace $ }}'
3637
g2pPbmsBgTaskClientSecret: ''
3738

3839
keymanagerBaseUrl: https://keymanager.openg2p.org
@@ -65,7 +66,7 @@ postgres-init:
6566
# TODO: Read variables from globals instead of hardcoding below
6667
- name: '{{ .Release.Name }}_bgtask_db'
6768
user: '{{ .Release.Name}}_bgtask_db_user'
68-
secret: '{{ .Release.Name }}_bgtask'
69+
secret: '{{ .Release.Name }}-bgtask'
6970
secretUserPasswordKey: '{{ .Release.Name}}_bgtask_db_user'
7071

7172
odoo:

0 commit comments

Comments
 (0)