This is an automated email from the ASF dual-hosted git repository.

dpgaspar pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git


The following commit(s) were added to refs/heads/master by this push:
     new 37909aa  feat: add possibility to set ingress class name in Helm chart 
(#17252)
37909aa is described below

commit 37909aace0b8968475c2c731891cd9dc52f8de85
Author: Jolcik <[email protected]>
AuthorDate: Thu Oct 28 16:11:10 2021 +0200

    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 <[email protected]>
---
 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 0c5b6a1..d62a0f5 100644
--- a/helm/superset/Chart.yaml
+++ b/helm/superset/Chart.yaml
@@ -22,7 +22,7 @@ maintainers:
   - name: craig-rueda
     email: [email protected]
     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 7c1dd72..2a151cc 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 72c515a..eeecea7 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"

Reply via email to