Merge pull request #46 from uc-cdis/feat/small_fixes

Fixing the dicom-server Helm chart
uc-cdis · Oct 18, 2022 · 88ddcfc · 88ddcfc
2 parents 1683c8d + 74df39a
commit 88ddcfc
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 2 deletions.
diff --git a/helm/dicom-server/templates/_helpers.tpl b/helm/dicom-server/templates/_helpers.tpl
@@ -60,4 +60,16 @@ Create the name of the service account to use
 {{- else }}
 {{- default "default" .Values.serviceAccount.name }}
 {{- end }}
+{{- end }}
+
+{{/*
+ Postgres Password lookup
+*/}}
+{{- define "dicom.postgres.password" -}}
+{{- $localpass := (lookup "v1" "Secret" "postgres" "postgres-postgresql" ) -}}
+{{- if $localpass }}
+{{- default (index $localpass.data "postgres-password" | b64dec) }}
+{{- else }}
+{{- default .Values.secrets.password }}
+{{- end }}
 {{- end }}
diff --git a/helm/dicom-server/templates/config-volume-g3auto.yaml b/helm/dicom-server/templates/config-volume-g3auto.yaml
@@ -8,7 +8,7 @@ stringData:
         {
           "db_host": {{ .Values.secrets.host | quote }},
           "db_username": {{ .Values.secrets.userName | quote }},
-          "db_password": {{ .Values.secrets.password | quote }},
+          "db_password": {{ include "dicom.postgres.password" . | quote }},
           "db_database": {{ .Values.secrets.dataBase | quote }}
         }
   orthanc_config_overwrites.json: |
@@ -21,7 +21,7 @@ stringData:
             "Host": {{ .Values.secrets.host | quote }},
             "Database": {{ .Values.secrets.dataBase | quote }},
             "Username": {{ .Values.secrets.userName | quote }},
-            "Password": {{ .Values.secrets.password | quote }},
+            "Password": {{ include "dicom.postgres.password" . | quote }},
             "IndexConnectionsCount": {{ .Values.secrets.indexConnectionsCount }},
             "Lock": {{ .Values.secrets.lock }}
           },