refactor(charts): cleanup unneeded version checks (#1424)

eclipse-tractusx · Jul 16, 2024 · 76b18db · 76b18db
1 parent c2e9506
commit 76b18db
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 80 deletions.
diff --git a/charts/tractusx-connector-azure-vault/templates/ingress-controlplane.yaml b/charts/tractusx-connector-azure-vault/templates/ingress-controlplane.yaml
@@ -20,33 +20,21 @@
 {{- $fullName := include "txdc.fullname" . }}
 {{- $controlLabels := include "txdc.controlplane.labels" . }}
 {{- $controlEdcEndpoints := .Values.controlplane.endpoints }}
-{{- $gitVersion := .Capabilities.KubeVersion.GitVersion }}
 {{- $namespace := .Release.Namespace }}
 
 {{- range .Values.controlplane.ingresses }}
 {{- if and .enabled .endpoints }}
 {{- $controlIngressName := printf "%s-controlplane-%s" $fullName .hostname }}
 {{- $annotations := .annotations | default dict }}
 ---
-{{- if semverCompare ">=1.19-0" $gitVersion }}
 apiVersion: networking.k8s.io/v1
-{{- else if semverCompare ">=1.14-0" $gitVersion }}
-apiVersion: networking.k8s.io/v1beta1
-{{- else }}
-apiVersion: extensions/v1beta1
-{{- end }}
 kind: Ingress
 metadata:
   name: {{ $controlIngressName }}
   namespace: {{ $namespace | default "default" | quote }}
   labels:
     {{- $controlLabels | nindent 4 }}
   annotations:
-    {{- if and .className (not (semverCompare ">=1.18-0" $gitVersion)) }}
-    {{- if not (hasKey $annotations "kubernetes.io/ingress.class") }}
-    {{- $_ := set $annotations "kubernetes.io/ingress.class" .className}}
-    {{- end }}
-    {{- end }}
     {{- if .certManager }}
     {{- if .certManager.issuer }}
     {{- $_ := set $annotations "cert-manager.io/issuer" .certManager.issuer}}
@@ -59,7 +47,7 @@ metadata:
     {{- toYaml . | nindent 4 }}
     {{- end }}
 spec:
-  {{- if and .className (semverCompare ">=1.18-0" $gitVersion) }}
+  {{- if .className }}
   ingressClassName: {{ .className }}
   {{- end }}
   {{- if .hostname }}
@@ -83,13 +71,10 @@ spec:
           - path: {{ $mapping.path }}
             pathType: Prefix
             backend:
-              {{- if semverCompare ">=1.19-0" $gitVersion }}
               service:
                 name: {{ $fullName }}-controlplane
                 port:
                   number: {{ $mapping.port }}
-              {{- else }}
-              {{- end }}
         {{- end }}
   {{- end }}
   {{- end }}

diff --git a/charts/tractusx-connector-azure-vault/templates/ingress-dataplane.yaml b/charts/tractusx-connector-azure-vault/templates/ingress-dataplane.yaml
@@ -20,33 +20,21 @@
 {{- $fullName := include "txdc.fullname" . }}
 {{- $dataLabels := include "txdc.dataplane.labels" . }}
 {{- $dataEdcEndpoints := .Values.dataplane.endpoints }}
-{{- $gitVersion := .Capabilities.KubeVersion.GitVersion }}
 {{- $namespace := .Release.Namespace }}
 
 {{- range .Values.dataplane.ingresses }}
 {{- if and .enabled .endpoints }}
 {{- $dataIngressName := printf "%s-dataplane-%s" $fullName .hostname }}
 {{- $annotations := .annotations | default dict }}
 ---
-{{- if semverCompare ">=1.19-0" $gitVersion }}
 apiVersion: networking.k8s.io/v1
-{{- else if semverCompare ">=1.14-0" $gitVersion }}
-apiVersion: networking.k8s.io/v1beta1
-{{- else }}
-apiVersion: extensions/v1beta1
-{{- end }}
 kind: Ingress
 metadata:
   name: {{ $dataIngressName }}
   namespace: {{ $namespace | default "default" | quote }}
   labels:
     {{- $dataLabels | nindent 4 }}
   annotations:
-    {{- if and .className (not (semverCompare ">=1.18-0" $gitVersion)) }}
-    {{- if not (hasKey $annotations "kubernetes.io/ingress.class") }}
-    {{- $_ := set $annotations "kubernetes.io/ingress.class" .className}}
-    {{- end }}
-    {{- end }}
     {{- if .certManager }}
     {{- if .certManager.issuer }}
     {{- $_ := set $annotations "cert-manager.io/issuer" .certManager.issuer}}
@@ -59,7 +47,7 @@ metadata:
     {{- toYaml . | nindent 4 }}
     {{- end }}
 spec:
-  {{- if and .className (semverCompare ">=1.18-0" $gitVersion) }}
+  {{- if .className }}
   ingressClassName: {{ .className }}
   {{- end }}
   {{- if .hostname }}
@@ -83,13 +71,10 @@ spec:
           - path: {{ $mapping.path }}
             pathType: Prefix
             backend:
-              {{- if semverCompare ">=1.19-0" $gitVersion }}
               service:
                 name: {{ $fullName }}-dataplane
                 port:
                   number: {{ $mapping.port }}
-              {{- else }}
-              {{- end }}
         {{- end }}
   {{- end }}
   {{- end }}

diff --git a/charts/tractusx-connector-memory/templates/ingress-runtime.yaml b/charts/tractusx-connector-memory/templates/ingress-runtime.yaml
@@ -24,33 +24,21 @@
 {{- $fullName := include "txdc.fullname" . }}
 {{- $controlLabels := include "txdc.runtime.labels" . }}
 {{- $controlEdcEndpoints := .Values.runtime.endpoints }}
-{{- $gitVersion := .Capabilities.KubeVersion.GitVersion }}
 {{- $namespace := .Release.Namespace }}
 
 {{- range .Values.runtime.ingresses }}
 {{- if and .enabled .endpoints }}
 {{- $controlIngressName := printf "%s-runtime-%s" $fullName .hostname }}
 {{- $annotations := .annotations | default dict }}
 ---
-{{- if semverCompare ">=1.19-0" $gitVersion }}
 apiVersion: networking.k8s.io/v1
-{{- else if semverCompare ">=1.14-0" $gitVersion }}
-apiVersion: networking.k8s.io/v1beta1
-{{- else }}
-apiVersion: extensions/v1beta1
-{{- end }}
 kind: Ingress
 metadata:
   name: {{ $controlIngressName }}
   namespace: {{ $namespace | default "default" | quote }}
   labels:
     {{- $controlLabels | nindent 4 }}
   annotations:
-    {{- if and .className (not (semverCompare ">=1.18-0" $gitVersion)) }}
-    {{- if not (hasKey $annotations "kubernetes.io/ingress.class") }}
-    {{- $_ := set $annotations "kubernetes.io/ingress.class" .className}}
-    {{- end }}
-    {{- end }}
     {{- if .certManager }}
     {{- if .certManager.issuer }}
     {{- $_ := set $annotations "cert-manager.io/issuer" .certManager.issuer}}
@@ -63,7 +51,7 @@ metadata:
     {{- toYaml . | nindent 4 }}
     {{- end }}
 spec:
-  {{- if and .className (semverCompare ">=1.18-0" $gitVersion) }}
+  {{- if .className }}
   ingressClassName: {{ .className }}
   {{- end }}
   {{- if .hostname }}
@@ -87,13 +75,10 @@ spec:
           - path: {{ $mapping.path }}
             pathType: Prefix
             backend:
-              {{- if semverCompare ">=1.19-0" $gitVersion }}
               service:
                 name: {{ $fullName }}-runtime
                 port:
                   number: {{ $mapping.port }}
-              {{- else }}
-              {{- end }}
         {{- end }}
   {{- end }}
   {{- end }}

diff --git a/charts/tractusx-connector/templates/ingress-controlplane.yaml b/charts/tractusx-connector/templates/ingress-controlplane.yaml
@@ -24,33 +24,21 @@
 {{- $fullName := include "txdc.fullname" . }}
 {{- $controlLabels := include "txdc.controlplane.labels" . }}
 {{- $controlEdcEndpoints := .Values.controlplane.endpoints }}
-{{- $gitVersion := .Capabilities.KubeVersion.GitVersion }}
 {{- $namespace := .Release.Namespace }}
 
 {{- range .Values.controlplane.ingresses }}
 {{- if and .enabled .endpoints }}
 {{- $controlIngressName := printf "%s-controlplane-%s" $fullName .hostname }}
 {{- $annotations := .annotations | default dict }}
 ---
-{{- if semverCompare ">=1.19-0" $gitVersion }}
 apiVersion: networking.k8s.io/v1
-{{- else if semverCompare ">=1.14-0" $gitVersion }}
-apiVersion: networking.k8s.io/v1beta1
-{{- else }}
-apiVersion: extensions/v1beta1
-{{- end }}
 kind: Ingress
 metadata:
   name: {{ $controlIngressName }}
   namespace: {{ $namespace | default "default" | quote }}
   labels:
     {{- $controlLabels | nindent 4 }}
   annotations:
-    {{- if and .className (not (semverCompare ">=1.18-0" $gitVersion)) }}
-    {{- if not (hasKey $annotations "kubernetes.io/ingress.class") }}
-    {{- $_ := set $annotations "kubernetes.io/ingress.class" .className}}
-    {{- end }}
-    {{- end }}
     {{- if .certManager }}
     {{- if .certManager.issuer }}
     {{- $_ := set $annotations "cert-manager.io/issuer" .certManager.issuer}}
@@ -63,7 +51,7 @@ metadata:
     {{- toYaml . | nindent 4 }}
     {{- end }}
 spec:
-  {{- if and .className (semverCompare ">=1.18-0" $gitVersion) }}
+  {{- if .className }}
   ingressClassName: {{ .className }}
   {{- end }}
   {{- if .hostname }}
@@ -87,13 +75,10 @@ spec:
           - path: {{ $mapping.path }}
             pathType: Prefix
             backend:
-              {{- if semverCompare ">=1.19-0" $gitVersion }}
               service:
                 name: {{ $fullName }}-controlplane
                 port:
                   number: {{ $mapping.port }}
-              {{- else }}
-              {{- end }}
         {{- end }}
   {{- end }}
   {{- end }}

diff --git a/charts/tractusx-connector/templates/ingress-dataplane.yaml b/charts/tractusx-connector/templates/ingress-dataplane.yaml
@@ -24,33 +24,21 @@
 {{- $fullName := include "txdc.fullname" . }}
 {{- $dataLabels := include "txdc.dataplane.labels" . }}
 {{- $dataEdcEndpoints := .Values.dataplane.endpoints }}
-{{- $gitVersion := .Capabilities.KubeVersion.GitVersion }}
 {{- $namespace := .Release.Namespace }}
 
 {{- range .Values.dataplane.ingresses }}
 {{- if and .enabled .endpoints }}
 {{- $dataIngressName := printf "%s-dataplane-%s" $fullName .hostname }}
 {{- $annotations := .annotations | default dict }}
 ---
-{{- if semverCompare ">=1.19-0" $gitVersion }}
 apiVersion: networking.k8s.io/v1
-{{- else if semverCompare ">=1.14-0" $gitVersion }}
-apiVersion: networking.k8s.io/v1beta1
-{{- else }}
-apiVersion: extensions/v1beta1
-{{- end }}
 kind: Ingress
 metadata:
   name: {{ $dataIngressName }}
   namespace: {{ $namespace | default "default" | quote }}
   labels:
     {{- $dataLabels | nindent 4 }}
   annotations:
-    {{- if and .className (not (semverCompare ">=1.18-0" $gitVersion)) }}
-    {{- if not (hasKey $annotations "kubernetes.io/ingress.class") }}
-    {{- $_ := set $annotations "kubernetes.io/ingress.class" .className}}
-    {{- end }}
-    {{- end }}
     {{- if .certManager }}
     {{- if .certManager.issuer }}
     {{- $_ := set $annotations "cert-manager.io/issuer" .certManager.issuer}}
@@ -63,7 +51,7 @@ metadata:
     {{- toYaml . | nindent 4 }}
     {{- end }}
 spec:
-  {{- if and .className (semverCompare ">=1.18-0" $gitVersion) }}
+  {{- if .className }}
   ingressClassName: {{ .className }}
   {{- end }}
   {{- if .hostname }}
@@ -87,13 +75,10 @@ spec:
           - path: {{ $mapping.path }}
             pathType: Prefix
             backend:
-              {{- if semverCompare ">=1.19-0" $gitVersion }}
               service:
                 name: {{ $fullName }}-dataplane
                 port:
                   number: {{ $mapping.port }}
-              {{- else }}
-              {{- end }}
         {{- end }}
   {{- end }}
   {{- end }}