Updating NOMINATIM_DATABASE_DSN format (#97)

* Updated DSN Format. Renaming postgres ENV variable for clarity * Reverting Chart.lock as that is not in question here. * Reverting change to Chart.yaml as that is also not in question
robjuz · Feb 1, 2025 · 18e22b9 · 18e22b9
1 parent e21a99d
commit 18e22b9
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 6 deletions.
diff --git a/charts/nominatim/templates/_helpers.tpl b/charts/nominatim/templates/_helpers.tpl
@@ -90,6 +90,13 @@ Create the database URL.
 postgresql://{{ include "nominatim.databaseUser" . }}:{{ include "nominatim.databasePassword" . }}@{{ include "nominatim.databaseHost" . }}:{{ include "nominatim.databasePort" . }}/{{ include "nominatim.databaseName" . }}
 {{- end }}
 
+{{/*
+Create the database DSN.
+*/}}
+{{- define "nominatim.databaseDSN" -}}
+pgsql:dbname={{ include "nominatim.databaseName" . }};host={{ include "nominatim.databaseHost" . }};port={{ include "nominatim.databasePort" . }};user={{ include "nominatim.databaseUser" . }};password={{ include "nominatim.databasePassword" . }}
+{{- end }}
+
 {{- define "nominatim.containerPort" -}}
 {{- ternary 80 8080 .Values.nominatimUi.enabled -}}
 {{- end }}

diff --git a/charts/nominatim/templates/deployment.yaml b/charts/nominatim/templates/deployment.yaml
@@ -162,7 +162,7 @@ spec:
                   name: {{ .Values.externalDatabase.existingSecretDsn }}
                   key: {{ .Values.externalDatabase.existingSecretDsnKey }}
               {{- else }}
-              value: {{ include "nominatim.databaseUrl" . }}
+              value: {{ include "nominatim.databaseDSN" . }}
               {{- end }}
             {{- if .Values.extraEnvVars }}
             {{- include "common.tplvalues.render" (dict "value" .Values.extraEnvVars "context" $) | nindent 12 }}

diff --git a/charts/nominatim/templates/initJob.yaml b/charts/nominatim/templates/initJob.yaml
@@ -29,7 +29,7 @@ spec:
         - name: wait-for-db
           image: postgres
           env:
-            - name: NOMINATIM_DATABASE_DSN
+            - name: NOMINATIM_DATABASE_URL
               {{- if .Values.externalDatabase.existingSecretDsn }}
               valueFrom:
                 secretKeyRef:
@@ -42,8 +42,8 @@ spec:
             - /bin/bash
             - -c
             - |
-              echo $NOMINATIM_DATABASE_DSN;
-              until pg_isready -d $NOMINATIM_DATABASE_DSN; do
+              echo $NOMINATIM_DATABASE_URL;
+              until pg_isready -d $NOMINATIM_DATABASE_URL; do
                 echo "Waiting for database..."
                 sleep 2
               done
@@ -128,7 +128,7 @@ spec:
                   name: {{ .Values.externalDatabase.existingSecretDsn }}
                   key: {{ .Values.externalDatabase.existingSecretDsnKey }}
               {{- else }}
-              value: {{ include "nominatim.databaseUrl" . }}
+              value: {{ include "nominatim.databaseDSN" . }}
               {{- end }}
             {{- if .Values.flatnode.enabled }}
             - name: NOMINATIM_FLATNODE_FILE

diff --git a/charts/nominatim/templates/updatesJob.yaml b/charts/nominatim/templates/updatesJob.yaml
@@ -113,7 +113,7 @@ spec:
                       key: {{ .Values.externalDatabase.existingSecretDsnKey }}
                   {{- end }}
                   {{- if not .Values.externalDatabase.existingSecretDsn }}
-                  value: {{ include "nominatim.databaseUrl" . }}
+                  value: {{ include "nominatim.databaseDSN" . }}
                   {{- end }}
                 - name: NOMINATIM_REPLICATION_URL
                   value: {{ .Values.updates.replicationUrl }}