Fix ingress compat (#29)

* fix ingress compatibility with different k8s version

* bump up version

charts/clearml/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: clearml
 description: MLOps platform
 type: application
-version: "3.0.1"
+version: "3.0.2"
 appVersion: "1.1.1"
 home: https://clear.ml
 icon: https://raw.githubusercontent.com/allegroai/clearml/master/docs/clearml-logo.svg

charts/clearml/README.md

@@ -1,6 +1,6 @@
 # ClearML Ecosystem for Kubernetes
 
-![Version: 3.0.1](https://img.shields.io/badge/Version-3.0.1-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 1.1.1](https://img.shields.io/badge/AppVersion-1.1.1-informational?style=flat-square)
+![Version: 3.0.2](https://img.shields.io/badge/Version-3.0.2-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: 1.1.1](https://img.shields.io/badge/AppVersion-1.1.1-informational?style=flat-square)
 
 MLOps platform
 

charts/clearml/templates/ingress-api.yaml

@@ -23,12 +23,20 @@ spec:
       secretName: {{ .Values.ingress.api.tlsSecretName }}
   {{- end }}
   rules:
-    - host: {{ .Values.ingress.api.hostName }}
+  - host: {{ .Values.ingress.api.hostName }}
-      http:
+    http:
-        paths:
+      paths:
-          - path: "/"
+      - path: "/"
-            pathType: Prefix
+  {{ if semverCompare ">=1.19-0" .Capabilities.KubeVersion.GitVersion }}
-            backend:
+        pathType: Prefix
-              serviceName: {{ include "clearml.fullname" . }}-apiserver
+        backend:
-              servicePort: {{ .Values.apiserver.service.port }}
+          service:
+            name: {{ include "clearml.fullname" . }}-apiserver
+            port:
+              number: {{ .Values.apiserver.service.port }}
+  {{ else }}
+        backend:
+          serviceName: {{ include "clearml.fullname" . }}-apiserver
+          servicePort: {{ .Values.apiserver.service.port }}
+  {{ end }}
 {{- end }}

charts/clearml/templates/ingress-app.yaml

@@ -23,12 +23,20 @@ spec:
       secretName: {{ .Values.ingress.app.tlsSecretName }}
   {{- end }}
   rules:
-    - host: {{ .Values.ingress.app.hostName }}
+  - host: {{ .Values.ingress.app.hostName }}
-      http:
+    http:
-        paths:
+      paths:
-          - path: "/"
+      - path: "/"
-            pathType: Prefix
+  {{ if semverCompare ">=1.19-0" .Capabilities.KubeVersion.GitVersion }}
-            backend:
+        pathType: Prefix
-              serviceName: {{ include "clearml.fullname" . }}-webserver
+        backend:
-              servicePort: {{ .Values.webserver.service.port }}
+          service:
+            name: {{ include "clearml.fullname" . }}-webserver
+            port:
+              number: {{ .Values.webserver.service.port }}
+  {{ else }}
+        backend:
+          serviceName: {{ include "clearml.fullname" . }}-webserver
+          servicePort: {{ .Values.webserver.service.port }}
+  {{ end }}
 {{- end }}

charts/clearml/templates/ingress-files.yaml

@@ -23,12 +23,20 @@ spec:
       secretName: {{ .Values.ingress.files.tlsSecretName }}
   {{- end }}
   rules:
-    - host: {{ .Values.ingress.files.hostName }}
+  - host: {{ .Values.ingress.files.hostName }}
-      http:
+    http:
-        paths:
+      paths:
-          - path: "/"
+      - path: "/"
-            pathType: Prefix
+  {{ if semverCompare ">=1.19-0" .Capabilities.KubeVersion.GitVersion }}
-            backend:
+        pathType: Prefix
-              serviceName: {{ include "clearml.fullname" . }}-fileserver
+        backend:
-              servicePort: {{ .Values.fileserver.service.port }}
+          service:
+            name: {{ include "clearml.fullname" . }}-fileserver
+            port:
+              number: {{ .Values.fileserver.service.port }}
+  {{ else }}
+        backend:
+          serviceName: {{ include "clearml.fullname" . }}-fileserver
+          servicePort: {{ .Values.fileserver.service.port }}
+  {{ end }}
 {{- end }}