From abed99b1abfd6e933fead69190d4482be19cadd7 Mon Sep 17 00:00:00 2001
From: OZGCloud <ozgcloud@mgm-tp.com>
Date: Mon, 7 Oct 2024 12:13:46 +0200
Subject: [PATCH] OZG-3034 OZG-6684 Do not generate alfaHost-env variable for
empty value
---
src/main/helm/templates/_helpers.tpl | 8 +---
src/main/helm/templates/deployment.yaml | 3 ++
.../helm/deployment_alfa_host_env_test.yaml | 46 -------------------
src/test/helm/deployment_alfa_host_test.yaml | 9 ++--
4 files changed, 8 insertions(+), 58 deletions(-)
delete mode 100644 src/test/helm/deployment_alfa_host_env_test.yaml
diff --git a/src/main/helm/templates/_helpers.tpl b/src/main/helm/templates/_helpers.tpl
index d109ea608..c87e1565c 100644
--- a/src/main/helm/templates/_helpers.tpl
+++ b/src/main/helm/templates/_helpers.tpl
@@ -135,11 +135,7 @@ app.kubernetes.io/namespace: {{ include "app.namespace" . }}
{{- define "app.alfaHost" -}}
{{- if .Values.alfaHost -}}
{{ .Values.alfaHost | quote }}
-{{- else -}}
-{{- $bezeichner := default "" (.Values.ozgcloud).bezeichner -}}
-{{- $baseUrl := default "" .Values.baseUrl -}}
-{{- if and $bezeichner $baseUrl -}}
-{{ print "https://" $bezeichner "." $baseUrl | quote }}
-{{- end -}}
+{{- else if and (.Values.ozgcloud).bezeichner .Values.baseUrl -}}
+{{ print "https://" (.Values.ozgcloud).bezeichner "." .Values.baseUrl | quote }}
{{- end -}}
{{- end -}}
diff --git a/src/main/helm/templates/deployment.yaml b/src/main/helm/templates/deployment.yaml
index 65b3f7d20..e5995fb3d 100644
--- a/src/main/helm/templates/deployment.yaml
+++ b/src/main/helm/templates/deployment.yaml
@@ -263,8 +263,11 @@ spec:
value: "false"
{{- end }}
+ {{- $alfaHost := (include "app.alfaHost" .) -}}
+ {{- if $alfaHost }}
- name: ozgcloud_alfa_host
value: {{ include "app.alfaHost" . }}
+ {{- end }}
- name: ozgcloud_vorgangManager_serviceAddress
value: {{ include "app.ozgcloud_vorgangmanager_address" . }}
diff --git a/src/test/helm/deployment_alfa_host_env_test.yaml b/src/test/helm/deployment_alfa_host_env_test.yaml
deleted file mode 100644
index 0570b2e53..000000000
--- a/src/test/helm/deployment_alfa_host_env_test.yaml
+++ /dev/null
@@ -1,46 +0,0 @@
-#
-# Copyright (C) 2024 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: deployment alfa host environment
-release:
- name: vorgang-manager
- namespace: by-helm-test
-templates:
- - templates/deployment.yaml
-set:
- ozgcloud:
- environment: dev
- imagePullSecret: test-image-pull-secret
-tests:
- - it: check if alfa host is set
- set:
- baseUrl: test.company.local
- ozgcloud:
- bezeichner: helm
- asserts:
- - contains:
- path: spec.template.spec.containers[0].env
- content:
- name: ozgcloud_alfa_host
- value: https://helm.test.company.local
\ No newline at end of file
diff --git a/src/test/helm/deployment_alfa_host_test.yaml b/src/test/helm/deployment_alfa_host_test.yaml
index b9882fad4..37a6fdd0b 100644
--- a/src/test/helm/deployment_alfa_host_test.yaml
+++ b/src/test/helm/deployment_alfa_host_test.yaml
@@ -59,30 +59,27 @@ tests:
ozgcloud:
bezeichner:
asserts:
- - contains:
+ - notContains:
path: spec.template.spec.containers[0].env
content:
name: ozgcloud_alfa_host
- value:
- it: should not set alfa host if base url is not set
set:
baseUrl:
ozgcloud:
bezeichner: helm
asserts:
- - contains:
+ - notContains:
path: spec.template.spec.containers[0].env
content:
name: ozgcloud_alfa_host
- value:
- it: should not set alfa host if both bezeichner and base url are not set
set:
baseUrl:
ozgcloud:
bezeichner:
asserts:
- - contains:
+ - notContains:
path: spec.template.spec.containers[0].env
content:
name: ozgcloud_alfa_host
- value:
\ No newline at end of file
--
GitLab