From 54505976548630c30890cf7091a522e3fdc2e0fa Mon Sep 17 00:00:00 2001
From: ABI CHAHINE <rachad.abi-chahine@hotmail.fr>
Date: Wed, 15 Sep 2021 02:13:04 +0200
Subject: [PATCH] docs: upgrade helm chart to take in consideration K8S v1.19+
 for ingress (#4437)

Co-authored-by: Rachad <rachad.abi-chahine@decathlon.com>

***NO_CI***
---
 dev/helm/templates/ingress.yaml | 35 ++++++++++++++++++++++++---------
 dev/helm/values.yaml            | 13 +++++++-----
 2 files changed, 34 insertions(+), 14 deletions(-)

diff --git a/dev/helm/templates/ingress.yaml b/dev/helm/templates/ingress.yaml
index 0b08134d..adf10b65 100644
--- a/dev/helm/templates/ingress.yaml
+++ b/dev/helm/templates/ingress.yaml
@@ -1,11 +1,18 @@
 {{- if .Values.ingress.enabled -}}
-{{- $fullName := include "wiki.fullname" . -}}
-{{- $svcPort := .Values.service.port -}}
-{{- if semverCompare ">=1.14-0" .Capabilities.KubeVersion.GitVersion -}}
+  {{- $fullName := include "wiki.fullname" . -}}
+  {{- $svcPort := .Values.service.port -}}
+  {{- if and .Values.ingress.className (not (semverCompare ">=1.18-0" .Capabilities.KubeVersion.GitVersion)) }}
+  {{- if not (hasKey .Values.ingress.annotations "kubernetes.io/ingress.class") }}
+  {{- $_ := set .Values.ingress.annotations "kubernetes.io/ingress.class" .Values.ingress.className}}
+  {{- end }}
+  {{- end }}
+  {{- if semverCompare ">=1.19-0" .Capabilities.KubeVersion.GitVersion -}}
+apiVersion: networking.k8s.io/v1
+  {{- else if semverCompare ">=1.14-0" .Capabilities.KubeVersion.GitVersion -}}
 apiVersion: networking.k8s.io/v1beta1
-{{- else -}}
+  {{- else -}}
 apiVersion: extensions/v1beta1
-{{- end }}
+  {{- end }}
 kind: Ingress
 metadata:
   name: {{ $fullName }}
@@ -31,11 +38,21 @@ spec:
     - host: {{ .host | quote }}
       http:
         paths:
-        {{- range .paths }}
-          - path: {{ . }}
+          {{- range .paths }}
+          - path: {{ .path }}
+            {{- if and .pathType (semverCompare ">=1.18-0" $.Capabilities.KubeVersion.GitVersion) }}
+            pathType: {{ .pathType }}
+            {{- end }}
             backend:
+              {{- if semverCompare ">=1.19-0" $.Capabilities.KubeVersion.GitVersion }}
+              service:
+                name: {{ $fullName }}
+                port:
+                  number: {{ $svcPort }}
+              {{- else }}
               serviceName: {{ $fullName }}
               servicePort: {{ $svcPort }}
-        {{- end }}
+              {{- end }}
+          {{- end }}
+  {{- end }}
   {{- end }}
-{{- end }}
diff --git a/dev/helm/values.yaml b/dev/helm/values.yaml
index 8a34f41c..bfa9fb3c 100644
--- a/dev/helm/values.yaml
+++ b/dev/helm/values.yaml
@@ -52,13 +52,16 @@ service:
   # annotations: {}
 
 ingress:
-  enabled: false
+  enabled: true
   annotations: {}
     # kubernetes.io/ingress.class: nginx
     # kubernetes.io/tls-acme: "true"
   hosts:
-    - host: wiki.local
-      paths: ["/"]
+    - host: wiki.minikube.local
+      paths:
+        - path: "/"
+          pathType: Prefix
+
   tls: []
   #  - secretName: chart-example-tls
   #    hosts:
@@ -92,7 +95,7 @@ postgresql:
   enabled: true
   ## ssl enforce SSL communication with PostgresSQL
   ## Default to false
-  ##  
+  ##
   # ssl: false
   ## ca Certificate of Authority
   ## Default to empty, point to location of CA
@@ -102,7 +105,7 @@ postgresql:
   ## Default to postgres
   ##
   # postgresqlHost: postgres
-  ## postgresqlPort port for postgres 
+  ## postgresqlPort port for postgres
   ## Default to 5432
   ##
   # postgresqlPort: 5432