feat(ui): extra envs

Resolves #40

templates/deployment.yaml

@@ -77,10 +77,13 @@ spec:
           {{- range .Values.ui.extraArgs }}
           - {{ . | quote }}
           {{- end }}
-          {{ if .Values.ui.consumerProperties }}
           env:
-            - name: CONSUMER_PROPERTIES_FILE
-              value:  /conf/consumer.properties
+          {{- if .Values.ui.consumerProperties }}
+          - name: CONSUMER_PROPERTIES_FILE
+            value:  /conf/consumer.properties
+          {{- end }}
+          {{- with .Values.ui.extraEnv -}}
+            {{  toYaml . | nindent 10 }}
           {{- end }}
           readinessProbe:
             httpGet:

test/linter/test.sh

@@ -55,3 +55,19 @@ if [ $? -eq 0 ]; then
   exit 1
 fi
 
+helm lint . --strict --set 'ui.extraEnv=false'
+if [ $? -eq 0 ]; then
+  exit 1
+fi
+
+helm lint . --strict --set 'ui.extraEnv={1,2,3}'
+if [ $? -eq 0 ]; then
+  exit 1
+fi
+
+helm lint . --strict --set 'ui.extraEnv[0].name=qwe' --set 'ui.extraEnv[0].value=asd'
+if [ $? -ne 0 ]; then
+  exit -1
+fi
+
+

test/unit/deployment.yaml

@@ -0,0 +1,22 @@
+suite: deployment
+templates:
+  - deployment.yaml
+tests:
+  - it: should emit empty env by default
+    asserts:
+      - isEmpty:
+          path: spec.template.spec.containers[1].env
+  - it: should populate env when passed
+    set:
+      ui:
+        extraEnv:
+          - name: myvar
+            value: myvalue
+    asserts:
+      - contains:
+          path: spec.template.spec.containers[1].env
+          content:
+            name: myvar
+            value: myvalue
+          count: 1
+          any: false

values.schema.json

@@ -153,10 +153,11 @@
       "type": "object", "title": "ui container k8s settings", "additionalProperties": false,
       "properties": {
         "port": {"type": "integer", "default": 9000},
-        "extraArgs": { "type": "array", "title": "extra cmd line arguments", "items": { "type": "string" }, "default": [] },
+        "extraArgs": { "type": "array", "title": "extra cmd line arguments", "items": {"type": "string"}, "default": [] },
         "resources": { "$ref":  "#/definitions/resources" },
         "consumerProperties": { "type": "object", "title": "provide key value base pairs for consumer properties according to java docs", "default": {} },
-        "consumerPropertiesSsl": { "$ref": "#/definitions/consumerSsl" }
+        "consumerPropertiesSsl": { "$ref": "#/definitions/consumerSsl" },
+        "extraEnv": { "type": "array", "title": "optional environment variables", "items": {"type": "object"}, "default": [] }
       }
     },
     "zk": {

values.yaml

@@ -93,6 +93,21 @@ ui:
     #   password: test123
     #   value: <some base64 encoded keystore>
 
+  # Additional environment variables (declared as k8s style)
+  # ref: https://kubernetes.io/docs/tasks/inject-data-application/define-environment-variable-container/
+  extraEnv: []
+    # - name: GREETING
+    #   value: "Warm greetings to"
+    # - name: SECRET_USERNAME
+    #   valueFrom:
+    #     secretKeyRef:
+    #       name: mysecret
+    #       key: username
+    # - name: SPECIAL_LEVEL_KEY
+    #   valueFrom:
+    #     configMapKeyRef:
+    #       name: special-config
+    #       key: SPECIAL_LEVEL
 
 # various settings for Zookeeper container
 zk: