Merge pull request #51 from northvolt/pc/data-config-through-secrets

Support setting dataSourceConfig through secrets or extraEnv

Author: RazcoDev

templates/deployment-client.yaml

@@ -45,7 +45,7 @@ spec:
           {{- if .Values.server }}
             - name: OPAL_SERVER_URL
               value: {{ printf "http://%s:%v" (include "opal.serverName" .) .Values.server.port | quote }}
-            {{- if not (or (.Values.server.dataConfigSources.external_source_url) (.Values.server.dataConfigSources.config) ) }}
+            {{- if not (or (.Values.server.dataConfigSources.external_source_url) (.Values.server.dataConfigSources.config) (hasKey .Values.server.extraEnv "OPAL_DATA_UPDATER_ENABLED") ) }}
             - name: OPAL_DATA_UPDATER_ENABLED
               value: "False"
             {{- end }}

templates/deployment-server.yaml

@@ -90,8 +90,10 @@ spec:
             {{- end }}
             - name: UVICORN_NUM_WORKERS
               value: {{ .Values.server.uvicornWorkers | quote }}
+            {{- if or .Values.server.dataConfigSources.config .Values.server.dataConfigSources.external_source_url }}
             - name: OPAL_DATA_CONFIG_SOURCES
               value: {{ .Values.server.dataConfigSources | toRawJson | squote }}
+            {{- end}}
             {{- if .Values.server.broadcastUri }}
             - name: OPAL_BROADCAST_URI
               value: {{ .Values.server.broadcastUri | quote }}

values.yaml

@@ -32,6 +32,9 @@ server:
     #     topics: ["policy_data"]
     #     dst_path: "/static"
 
+    # Option #4 - Leave config empty and instead supply using the OPAL_DATA_CONFIG_SOURCES environment variable through env or secret
+    # config: null
+
   broadcastUri: null
   broadcastPgsql: true
   uvicornWorkers: 4