From 37909aace0b8968475c2c731891cd9dc52f8de85 Mon Sep 17 00:00:00 2001 From: Jolcik Date: Thu, 28 Oct 2021 16:11:10 +0200 Subject: [PATCH] feat: add possibility to set ingress class name in Helm chart (#17252) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit * add possibility to set ingress class name * remove ingress class annotation comment Co-authored-by: Piotr KostrzeĊ„ski --- helm/superset/Chart.yaml | 2 +- helm/superset/templates/ingress.yaml | 3 +++ helm/superset/values.yaml | 2 +- 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/helm/superset/Chart.yaml b/helm/superset/Chart.yaml index 0c5b6a15af60..d62a0f5b6ee5 100644 --- 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 index 7c1dd72f0ebe..2a151ccc2e3e 100644 --- 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 index 72c515aa3a61..eeecea7d1705 100644 --- 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"