Merge pull request 'OZG-3034 Environment variables for notification-manager'...

Merge pull request 'OZG-3034 Environment variables for notification-manager' (#459) from OZG-3034-Erweiterung-des-Mailtexts-fuer-Benachrichtigungen into master

Reviewed-on: https://git.ozg-sh.de/ozgcloud-app/vorgang-manager/pulls/459


Reviewed-by: OZGCloud <ozgcloud@mgm-tp.com>

src/main/helm/templates/_helpers.tpl

@@ -50,7 +50,7 @@ app.kubernetes.io/namespace: {{ include "app.namespace" . }}
 {{- end -}}
 
 {{- define "app.ozgcloudEnvironment" -}}
-{{- required "ozgcloud.environment muss angegeben sein" (.Values.ozgcloud).environment -}}
+{{- required "ozgcloud.environment must be set" (.Values.ozgcloud).environment -}}
 {{- end -}}
 
 {{- define "app.vorgangManagerDatabase" -}}
@@ -131,3 +131,19 @@ app.kubernetes.io/namespace: {{ include "app.namespace" . }}
 {{- define "app.elsterTransferOperatorNamespace" -}}
 {{- required "elsterTransferOperator.namespace must be set" (.Values.elsterTransferOperator).namespace -}}
 {{- end -}}
+
+{{- define "app.ozgcloudBezeichner" -}}
+{{- required "ozgcloud.bezeichner must be set" (.Values.ozgcloud).bezeichner -}}
+{{- end -}}
+
+{{- define "app.baseUrl" -}}
+{{- required "baseUrl must be set" .Values.baseUrl }}
+{{- end -}}
+
+{{- define "app.alfaHost" -}}
+{{- if .Values.alfaHost -}}
+{{ .Values.alfaHost | quote }}
+{{- else -}}
+{{ printf "https://%s.%s" (include "app.ozgcloudBezeichner" .) (include "app.baseUrl" .) }}
+{{- end -}}
+{{- end -}}

src/main/helm/templates/deployment.yaml

@@ -263,6 +263,11 @@ spec:
             value: "false"
           {{- end }}
 
+          - name: ozgcloud_alfa_host
+            value: {{ include "app.alfaHost" . }}
+          - name: ozgcloud_vorgangManager_serviceAddress
+            value: {{ include "app.ozgcloud_vorgangmanager_address" . }}
+
         {{- if or (.Values.database).useExternal (.Values.ozgcloudProxyApi).apikey }}
         envFrom:
           {{- if (.Values.database).useExternal }}

src/main/helm/values.yaml

@@ -24,8 +24,6 @@
 
 cluster_env: ""
 
-baseUrl: test.sh.ozg-cloud.de
-
 replicaCount: 2
 
 # ozgcloud:

src/test/helm-linter-values.yaml

@@ -22,8 +22,12 @@
 # unter der Lizenz sind dem Lizenztext zu entnehmen.
 #
 
+
+baseUrl: test.sh.ozg-cloud.de
+
 ozgcloud:
   environment: test
+  bezeichner: helm
   infoManager:
     address: https://info-manager.my-wonderful-domain.local:9000
 

src/test/helm/deployment_63_chars_test.yaml

@@ -33,8 +33,10 @@ chart:
 templates:
   - templates/deployment.yaml
 set:
+  baseUrl: test.sh.ozg-cloud.de
   ozgcloud:
     environment: dev
+    bezeichner: helm
   imagePullSecret: test-image-pull-secret
 
 tests:

src/test/helm/deployment_actuator_test.yaml

@@ -29,8 +29,10 @@ release:
 templates:
   - templates/deployment.yaml
 set:
+  baseUrl: test.sh.ozg-cloud.de
   ozgcloud:
     environment: dev
+    bezeichner: helm
   imagePullSecret: test-image-pull-secret
 tests:
   - it: check readiness and startup probe

src/test/helm/deployment_administration_address_env_test.yaml

@@ -29,9 +29,11 @@ release:
 templates:
   - templates/deployment.yaml
 set:
+  baseUrl: test.sh.ozg-cloud.de
   ozgcloud:
     environment: dev
-  imagePullSecret: image-pull-secret
+    bezeichner: helm
+  imagePullSecret: test-image-pull-secret
 tests:
   - it: should have set administration server address
     set:

src/test/helm/deployment_alfa_host_test.yaml

+#
+# Copyright (C) 2022 Das Land Schleswig-Holstein vertreten durch den
+# Ministerpräsidenten des Landes Schleswig-Holstein
+# Staatskanzlei
+# Abteilung Digitalisierung und zentrales IT-Management der Landesregierung
+#
+# Lizenziert unter der EUPL, Version 1.2 oder - sobald
+# diese von der Europäischen Kommission genehmigt wurden -
+# Folgeversionen der EUPL ("Lizenz");
+# Sie dürfen dieses Werk ausschließlich gemäß
+# dieser Lizenz nutzen.
+# Eine Kopie der Lizenz finden Sie hier:
+#
+# https://joinup.ec.europa.eu/collection/eupl/eupl-text-eupl-12
+#
+# Sofern nicht durch anwendbare Rechtsvorschriften
+# gefordert oder in schriftlicher Form vereinbart, wird
+# die unter der Lizenz verbreitete Software "so wie sie
+# ist", OHNE JEGLICHE GEWÄHRLEISTUNG ODER BEDINGUNGEN -
+# ausdrücklich oder stillschweigend - verbreitet.
+# Die sprachspezifischen Genehmigungen und Beschränkungen
+# unter der Lizenz sind dem Lizenztext zu entnehmen.
+#
+
+suite: test alfa host
+release:
+  name: vorgang-manager
+  namespace: sh-helm-test
+templates:
+  - templates/deployment.yaml
+set:
+  ozgcloud:
+    environment: dev
+  imagePullSecret: test-image-pull-secret
+tests:
+  - it: should set the alfa host
+    set:
+      baseUrl: test.sh.ozg-cloud.de
+      ozgcloud:
+        bezeichner: helm
+    asserts:
+      - contains:
+          path: spec.template.spec.containers[0].env
+          content:
+            name: ozgcloud_alfa_host
+            value: https://helm.test.sh.ozg-cloud.de
+  - it: should use alfa host from values
+    set:
+      baseUrl: test.sh.ozg-cloud.de
+      ozgcloud:
+        bezeichner: helm
+      alfaHost: https://hamburg.mein-kreis.de
+    asserts:
+      - contains:
+          path: spec.template.spec.containers[0].env
+          content:
+            name: ozgcloud_alfa_host
+            value: https://hamburg.mein-kreis.de
+  - it: should fail if bezeichner is not set
+    set:
+      baseUrl: test.sh.ozg-cloud.de
+    asserts:
+      - failedTemplate:
+          errorMessage: "ozgcloud.bezeichner must be set"
+  - it: should fail if base url is not set
+    set:
+      ozgcloud:
+        bezeichner: helm
+    asserts:
+      - failedTemplate:
+          errorMessage: "baseUrl must be set"

src/test/helm/deployment_antragraum_test.yaml

@@ -29,9 +29,11 @@ release:
 templates:
   - templates/deployment.yaml
 set:
-  imagePullSecret: image-pull-secret
+  baseUrl: test.sh.ozg-cloud.de
   ozgcloud:
     environment: dev
+    bezeichner: helm
+  imagePullSecret: test-image-pull-secret
 tests:
   - it: should set antragsraum values
     set:

src/test/helm/deployment_bayernid_test.yaml

@@ -29,8 +29,10 @@ release:
 templates:
   - templates/deployment.yaml
 set:
+  baseUrl: test.sh.ozg-cloud.de
   ozgcloud:
     environment: dev
+    bezeichner: helm
   imagePullSecret: test-image-pull-secret
 tests:
   - it: should set bayernid values

src/test/helm/deployment_bindings_test.yaml

@@ -29,8 +29,10 @@ release:
 templates:
   - templates/deployment.yaml
 set:
+  baseUrl: test.sh.ozg-cloud.de
   ozgcloud:
     environment: dev
+    bezeichner: helm
   imagePullSecret: test-image-pull-secret
 tests:
   - it: should have volumes

src/test/helm/deployment_container_basic_test.yaml

@@ -29,8 +29,10 @@ release:
 templates:
   - templates/deployment.yaml
 set:
+  baseUrl: test.sh.ozg-cloud.de
   ozgcloud:
     environment: dev
+    bezeichner: helm
   imagePullSecret: test-image-pull-secret
 tests:
   - it: should have correct values for container image, name, imagePullPolicy

src/test/helm/deployment_container_security_context_test.yaml

@@ -29,8 +29,10 @@ release:
 templates:
   - templates/deployment.yaml
 set:
+  baseUrl: test.sh.ozg-cloud.de
   ozgcloud:
     environment: dev
+    bezeichner: helm
   imagePullSecret: test-image-pull-secret
 tests:
   - it: check default values

src/test/helm/deployment_defaults_labels_test.yaml

@@ -29,8 +29,10 @@ release:
 templates:
   - templates/deployment.yaml
 set:
+  baseUrl: test.sh.ozg-cloud.de
   ozgcloud:
     environment: dev
+    bezeichner: helm
   imagePullSecret: test-image-pull-secret
 tests:
   - it: check default labels

src/test/helm/deployment_defaults_topologySpreadConstraints_test.yaml

@@ -29,8 +29,10 @@ release:
 templates:
   - templates/deployment.yaml
 set:
+  baseUrl: test.sh.ozg-cloud.de
   ozgcloud:
-    environment: test
+    environment: dev
+    bezeichner: helm
   imagePullSecret: test-image-pull-secret
  
 tests:

src/test/helm/deployment_dummy_bescheid_document_test.yaml

@@ -29,12 +29,14 @@ release:
 templates:
   - templates/deployment.yaml
 set:
-  imagePullSecret: image-pull-secret
+  baseUrl: test.sh.ozg-cloud.de
   ozgcloud:
     environment: dev
+    bezeichner: helm
     feature:
       bescheid:
         enableDummyDocumentProcessor: true
+  imagePullSecret: image-pull-secret
 tests:
   - it: should enable dummy bescheid document processor
     asserts:

src/test/helm/deployment_elasticsearch_test.yaml

@@ -26,8 +26,10 @@ suite: elasticsearch tests
 templates:
   - templates/deployment.yaml
 set:
+  baseUrl: test.sh.ozg-cloud.de
   ozgcloud:
     environment: dev
+    bezeichner: helm
   imagePullSecret: test-image-pull-secret
 tests:
   - it: check if elastic credentials are disabled by default

src/test/helm/deployment_env_test.yaml

@@ -26,8 +26,10 @@ suite: test environments
 templates:
   - templates/deployment.yaml
 set:
+  baseUrl: test.sh.ozg-cloud.de
   ozgcloud:
     environment: dev
+    bezeichner: helm
   imagePullSecret: test-image-pull-secret
 tests:
   - it: check customList as list

src/test/helm/deployment_grpc_info_manager_address_test.yaml

@@ -28,14 +28,18 @@ release:
   namespace: sh-helm-test
 templates:
   - templates/deployment.yaml
+set:
+  baseUrl: test.sh.ozg-cloud.de
+  ozgcloud:
+    environment: dev
+    bezeichner: helm
+  imagePullSecret: test-image-pull-secret
 tests:
   - it: should set the grpc info-manager client address
     set:
       ozgcloud:
-        environment: dev
         infoManager:
           address: https://info-manager.my-wonderful-domain.local:9000
-      imagePullSecret: test-image-pull-secret
     asserts:
       - contains:
           path: spec.template.spec.containers[0].env