feat: add possibility to set ingress class name in Helm chart (#17252)

* add possibility to set ingress class name * remove ingress class annotation comment Co-authored-by: Piotr Kostrzeński <piotr.kostrzenski@samba.tv>
apache · Oct 28, 2021 · 37909aa · 37909aa
1 parent be6cca5
commit 37909aa
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/helm/superset/Chart.yaml b/helm/superset/Chart.yaml
@@ -22,7 +22,7 @@ maintainers:
   - name: craig-rueda
     email: craig@craigrueda.com
     url: https://github.com/craig-rueda
-version: 0.3.11
+version: 0.3.12
 dependencies:
 - name: postgresql
   version: 10.2.0

diff --git a/helm/superset/templates/ingress.yaml b/helm/superset/templates/ingress.yaml
@@ -30,6 +30,9 @@ metadata:
 {{ toYaml . | indent 4 }}
 {{- end }}
 spec:
+{{- if .Values.ingress.ingressClassName }}
+  ingressClassName: {{ .Values.ingress.ingressClassName }}
+{{- end }}
 {{- if .Values.ingress.tls }}
   tls:
   {{- range .Values.ingress.tls }}

diff --git a/helm/superset/values.yaml b/helm/superset/values.yaml
@@ -165,8 +165,8 @@ service:
 
 ingress:
   enabled: false
+  # ingressClassName: nginx
   annotations: {}
-    # kubernetes.io/ingress.class: nginx
     # kubernetes.io/tls-acme: "true"
     ## Extend timeout to allow long running queries.
     # nginx.ingress.kubernetes.io/proxy-connect-timeout: "300"