Merge pull request 'OZG-3363-helm-chart-anpassungen' (#62) from...

Merge pull request 'OZG-3363-helm-chart-anpassungen' (#62) from OZG-3363-helm-chart-anpassungen into master

Reviewed-on: https://git.ozg-sh.de/mgm/intelliform-adapter/pulls/62

xta-adapter/src/main/helm/templates/xta_adapter_cronjob.yaml

@@ -46,7 +46,7 @@ spec:
           restartPolicy: Never
           containers:
             - name: xta-adapter
-              image: "{{ .Values.image.repo }}/{{ .Values.image.name }}:{{ coalesce (.Values.image).tag "latest" }}"
+              image: "{{ .Values.image.repo }}/{{ .Values.image.name }}:{{ (.Values.image).tag }}"
               imagePullPolicy: Always
               env:
                 - name: spring_profiles_active
@@ -94,6 +94,50 @@ spec:
                   mountPath: "/workspace/keystore/xta-keystore.p12"
                   subPath: file
                   readOnly: true
+              securityContext:
+                allowPrivilegeEscalation: false
+                privileged: false
+                readOnlyRootFilesystem: false
+                runAsNonRoot: false
+              resources:
+              {{- with .Values.resources }}
+{{ toYaml . | indent 16 }}
+              {{- end }}
+
+              {{- if (.Values.dummyProbesEnabled) }}
+              livenessProbe:
+                exec:
+                  command:
+                  - echo
+                  - '>'
+                  - /dev/null
+                failureThreshold: 3
+                periodSeconds: 1
+                successThreshold: 1
+                timeoutSeconds: 1
+              readinessProbe:
+                exec:
+                  command:
+                  - echo
+                  - ""
+                  - '>'
+                  - /dev/null
+                failureThreshold: 3
+                periodSeconds: 1
+                successThreshold: 1
+                timeoutSeconds: 1
+              startupProbe:
+                exec:
+                  command:
+                  - echo
+                  - '>'
+                  - /dev/null
+                failureThreshold: 3
+                periodSeconds: 1
+                successThreshold: 1
+                timeoutSeconds: 1
+              {{- end }}
+
           volumes:
             - name: bindings
               configMap:

xta-adapter/src/main/helm/values.yaml

@@ -25,7 +25,7 @@
 image:
   repo: docker.ozg-sh.de
   name: xta-adapter
-  tag: latest
+  tag: 9.9.99
 
 # env:
 #   overrideSpringProfiles: "oc,prod"

xta-adapter/src/test/helm/xta_adapter_cronjob_basic_test.yaml

@@ -75,3 +75,28 @@ tests:
       - equal:
           path: spec.schedule
           value: "*/15 * * * *"
+  - it: check container image
+    template: xta_adapter_cronjob.yaml
+    set:
+      image.name: xta-adapter
+      kop.environment: dev
+    asserts:
+    - equal:
+        path: spec.jobTemplate.spec.template.spec.containers[0].image
+        value: "docker.ozg-sh.de/xta-adapter:9.9.99"
+
+  - it: check securityContext
+    template: xta_adapter_cronjob.yaml
+    asserts:
+    - equal:
+        path: spec.jobTemplate.spec.template.spec.containers[0].securityContext.allowPrivilegeEscalation
+        value: false
+    - equal:
+        path: spec.jobTemplate.spec.template.spec.containers[0].securityContext.privileged
+        value: false
+    - equal:
+        path: spec.jobTemplate.spec.template.spec.containers[0].securityContext.readOnlyRootFilesystem
+        value: false
+    - equal:
+        path: spec.jobTemplate.spec.template.spec.containers[0].securityContext.runAsNonRoot
+        value: false
\ No newline at end of file

xta-adapter/src/test/helm/xta_adapter_cronjob_dummy_probes_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 xta adapter cronjob
+templates:
+  - templates/xta_adapter_cronjob.yaml
+release:
+  name: xta-adapter
+  namespace: helm-test
+tests:
+  - it: check dummy livenessProbe default disabled
+    template: xta_adapter_cronjob.yaml
+    asserts:
+      - isEmpty:
+          path: spec.jobTemplate.spec.template.spec.containers[0].livenessProbe
+  - it: check dummy startupProbe default disabled
+    template: xta_adapter_cronjob.yaml
+    asserts:
+      - isEmpty:
+          path: spec.jobTemplate.spec.template.spec.containers[0].startupProbe
+  - it: check dummy readynessProbe default disabled
+    template: xta_adapter_cronjob.yaml
+    asserts:
+      - isEmpty:
+          path: spec.jobTemplate.spec.template.spec.containers[0].readinessProbe
+
+  - it: check dummy livenessProbe disabled
+    template: xta_adapter_cronjob.yaml
+    set:
+      dummyProbesEnabled: false
+    asserts:
+      - isEmpty:
+          path: spec.jobTemplate.spec.template.spec.containers[0].livenessProbe
+  - it: check dummy startupProbe disabled
+    template: xta_adapter_cronjob.yaml
+    set:
+      dummyProbesEnabled: false
+    asserts:
+      - isEmpty:
+          path: spec.jobTemplate.spec.template.spec.containers[0].startupProbe
+  - it: check dummy readynessProbe disabled
+    template: xta_adapter_cronjob.yaml
+    set:
+      dummyProbesEnabled: false
+    asserts:
+      - isEmpty:
+          path: spec.jobTemplate.spec.template.spec.containers[0].readinessProbe
+
+
+  - it: check dummy livenessProbe enabled
+    template: xta_adapter_cronjob.yaml
+    set:
+      dummyProbesEnabled: true
+    asserts:
+      - isNotEmpty:
+          path: spec.jobTemplate.spec.template.spec.containers[0].livenessProbe
+  - it: check dummy startupProbe enabled
+    template: xta_adapter_cronjob.yaml
+    set:
+      dummyProbesEnabled: true
+    asserts:
+      - isNotEmpty:
+          path: spec.jobTemplate.spec.template.spec.containers[0].startupProbe
+  - it: check dummy readynessProbe enabled
+    template: xta_adapter_cronjob.yaml
+    set:
+      dummyProbesEnabled: true
+    asserts:
+      - isNotEmpty:
+          path: spec.jobTemplate.spec.template.spec.containers[0].readinessProbe

xta-adapter/src/test/helm/xta_adapter_cronjob_resources_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 xta adapter cronjob
+templates:
+  - templates/xta_adapter_cronjob.yaml
+release:
+  name: xta-adapter
+  namespace: helm-test
+tests:
+  - it: check resources
+    template: xta_adapter_cronjob.yaml
+    set:
+      resources:
+        limits:
+          cpu: "11m"
+          memory: "22Mi"
+        requests:
+          cpu: "33m"
+          memory: "44Mi"
+    asserts:
+      - equal:
+          path: spec.jobTemplate.spec.template.spec.containers[0].resources.limits.cpu
+          value: "11m"
+      - equal:
+          path: spec.jobTemplate.spec.template.spec.containers[0].resources.limits.memory
+          value: "22Mi"
+      - equal:
+          path: spec.jobTemplate.spec.template.spec.containers[0].resources.requests.cpu
+          value: "33m"
+      - equal:
+          path: spec.jobTemplate.spec.template.spec.containers[0].resources.requests.memory
+          value: "44Mi"