Skip to content

Commit 6358a6f

Browse files
committed
[G2P-3424] WIP. Defining global variables.
1 parent bc405d4 commit 6358a6f

File tree

2 files changed

+60
-21
lines changed

2 files changed

+60
-21
lines changed

charts/openg2p-pbms/templates/_helper.tpl

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,29 @@
1+
{{/*
2+
Overriding Odoo's templates. All the variable names here match ones in Odoo's
3+
values.yaml, not this chart's values.yaml. The templates here will be available
4+
to Odoo's chart.
5+
*/}}
6+
7+
{{- define "odoo.databaseHost" -}}
8+
{{- tpl .Values.externalDatabase.host . -}}
9+
{{- end -}}
10+
11+
{{- define "odoo.databaseName" -}}
12+
{{- tpl .Values.externalDatabase.database . -}}
13+
{{- end -}}
14+
15+
{{- define "odoo.databaseUser" -}}
16+
{{- tpl .Values.externalDatabase.user . -}}
17+
{{- end -}}
18+
19+
{{- define "odoo.databaseSecretPasswordKey" -}}
20+
{{- tpl .Values.externalDatabase.existingSecretPasswordKey . -}}
21+
{{- end -}}
22+
23+
{{- define "odoo.databaseSecretName" -}}
24+
{{- tpl .Values.externalDatabase.existingSecret . -}}
25+
{{- end -}}
26+
127
{{/*
228
Expand the name of the chart.
329
*/}}

charts/openg2p-pbms/values.yaml

Lines changed: 34 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,22 @@ global:
33
keycloakIssuerUrl: '{{ tpl .Values.global.keycloakBaseUrl $ }}/realms/master'
44

55
postgresqlHost: openg2p-commons-postgresql
6-
opensearchInstallationName: openg2p-commons-opensearch
7-
reportingInstallationName: openg2p-commons-reporting
6+
7+
pbmsDB: '{{ .Release.Name }}_db'
8+
pbmsDBUser: '{{ .Release.Name}}_db_user'
9+
pbmsDBSecret: '{{ .Release.Name }}'
10+
pbmsDBUserPasswordKey: '{{ .Release.Name}}_db_user'
11+
12+
pbmsBgTaskDB: '{{ .Release.Name }}_bgtask_db'
13+
pbmsBgTaskDBUser: '{{ .Release.Name}}_bgtask_db_user'
14+
pbmsBgTaskDBSecret: '{{ .Release.Name }}_bgtask'
15+
pbmsBgTaskDBUserPasswordKey: '{{ .Release.Name}}_bgtask_db_user'
16+
17+
# Defaults that should be overridden by questions.yaml
18+
registryDB: 'registry_db'
19+
registryDBUser: 'registry_db_user'
20+
registryDBSecret: 'registry'
21+
registryDBUserPasswordKey: 'registry_db_user'
822

923
mailInstallationName: openg2p-commons-mail
1024

@@ -43,13 +57,13 @@ postgres-init:
4357
existingSecret: openg2p-commons-postgresql
4458
existingSecretPostgresPasswordKey: postgres-password
4559
databases:
46-
# Same as db in Odoo - make sure any changes to the name are synced with Odoo definitions
47-
- name: '{{ .Release.Name }}_db'
48-
user: '{{ .Release.Name}}_db_user'
60+
# Same as db in Odoo
61+
- name: '{{ tpl .Values.global.pbmsDB $ }}'
62+
user: '{{ tpl. .Values.global.pbmsDBUser $ }}'
4963
# Same as odoo.externalDatabase.existingSecret - make sure these two are same if any changes are made.
50-
secret: '{{ .Release.Name }}'
64+
secret: '{{ tpl .Values.global.pbmsDBSecret $ }}'
5165
# As convention same as user
52-
secretUserPasswordKey: '{{ .Release.Name}}_db_user'
66+
secretUserPasswordKey: '{{ tpl .Values.global.pbmsDBUserPasswordKey $ }}'
5367
- name: '{{ .Release.Name }}_bgtask_db'
5468
user: '{{ .Release.Name}}_bgtask_db_user'
5569
secret: '{{ .Release.Name }}_bgtask'
@@ -78,30 +92,28 @@ odoo:
7892

7993
postgresql:
8094
enabled: false
81-
#database: pbmsdb
82-
# CREATE DATABASE bgtaskdb;
8395

8496
externalDatabase:
8597
create: true
8698
# NOTE: For few parameter below the Odoo templates have been overridden to accomodate
8799
# templating. See templates/_helpers.tpl
88100
host: '{{ .Values.global.postgresqlHost }}'
89101
port: 5432
90-
database: '{{ .Release.Name }}_db'
91-
user: '{{ .Release.Name }}_db_user'
102+
database: '{{ .Values.global.pbmsDB }}'
103+
user: '{{ .Values.global.pbmsDBUser }}'
92104
# Provided via secrets below
93105
password: ""
94106
postgresqlPostgresUser: "postgres"
95107
# Passed via secrets
96108
postgresqlPostgresPassword: ""
97109
# The secrets below should be created before this installation is done.
98110
# Secret for the DB (not Postgresql server)
99-
existingSecret: '{{ .Release.Name }}'
111+
existingSecret: '{{ .Values.global.pbmsDBSecret }}'
100112
# This is secret for PostgreSQL server. Both have different secrets (unlike original Odoo
101113
# Chart where the same secret contained keys for both Postgres and the database user
102114
existingPostgresSecret: openg2p-commons-postgresql
103115
# Same as user name as convention
104-
existingSecretPasswordKey: '{{ .Release.Name }}_db_user'
116+
existingSecretPasswordKey: '{{ .Values.global.pbmsDBUserPasswordKey }}'
105117
# Hardcoded as this is not likely to change frequently
106118
existingSecretPostgresPasswordKey: postgres-password
107119

@@ -135,17 +147,18 @@ openg2p-pbms-bg-task-api:
135147
enabled: true
136148
nameOverride: api
137149

138-
keymanagerInstallationName: '{{ .Release.Name }}-keymanager'
150+
keymanagerInstallationName: '{{ tpl .Values.global.keymanagerInstallationName $ }}'
151+
139152
envVars:
140-
BG_TASK_API_DB_USERNAME_SR: 'postgres'
141-
BG_TASK_API_DB_HOSTNAME_SR: '{{ tpl .Values.global.registryInstallationName $ }}-postgresql'
153+
BG_TASK_API_DB_HOSTNAME_SR: '{{ tpl .Values.global.postgresqlHost $ }}'
142154
BG_TASK_API_DB_PORT_SR: 5432
143-
BG_TASK_API_DB_DBNAME_SR: 'socialregistrydb'
155+
BG_TASK_API_DB_DBNAME_SR: '{{ tpl .Values.global.registryDB $ }}'
156+
BG_TASK_API_DB_USERNAME_SR: '{{ tpl .Values.global.registryDBUser }}'
144157

145-
BG_TASK_API_DB_USERNAME: 'postgres'
146-
BG_TASK_API_DB_HOSTNAME: '{{ tpl .Values.postgresqlInstallationName $ }}'
158+
BG_TASK_API_DB_HOSTNAME: '{{ tpl .Values.global.postgresqlHost $ }}'
147159
BG_TASK_API_DB_PORT: 5432
148-
BG_TASK_API_DB_DBNAME: 'bgtaskdb'
160+
BG_TASK_API_DB_DBNAME: '{{ tpl .Values.global.pbmsBgTaskDB $ }}'
161+
BG_TASK_API_DB_USERNAME: '{{ tpl .Values.global.pbmsBgTaskDBUser $ }}'
149162

150163
openg2p-pbms-bg-task-celery-beat-producers:
151164
enabled: true
@@ -182,7 +195,7 @@ openg2p-pbms-bg-task-celery-workers:
182195
enabled: true
183196
nameOverride: celery-worker
184197

185-
keymanagerInstallationName: '{{ .Release.Name }}-keymanager'
198+
keymanagerInstallationName: '{{ tpl .Values.global.keymanagerInstallationName $ }}'
186199

187200
envVars:
188201
BG_TASK_CELERY_WORKERS_G2P_BRIDGE_BASE_URL: "https://g2pbridge-api"

0 commit comments

Comments
 (0)