Merge pull request #17 from InseeFrLab/superset-add-cert-manager

superset- add use of certmanager
InseeFrLab · Feb 8, 2024 · 45edcd3 · 45edcd3
2 parents 3f9e029 + 4c7649f
commit 45edcd3
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 1 deletion.
diff --git a/charts/superset/Chart.yaml b/charts/superset/Chart.yaml
@@ -33,5 +33,5 @@ dependencies:
     version: 0.10.11
     repository:  https://apache.github.io/superset
   - name: library-chart
-    version: 1.5.15
+    version: 1.5.19
     repository: https://inseefrlab.github.io/helm-charts-interactive-services
diff --git a/charts/superset/templates/ingress.yaml b/charts/superset/templates/ingress.yaml
@@ -16,6 +16,9 @@ spec:
   tls:
     - hosts:
         - {{ .Values.ingress.hostname | quote }}
+    {{- if .Values.ingress.useCertManager }}
+      secretName: tls-cert-{{ include "library-chart.fullname" . }}
+    {{- end }}
 {{- end }}
   rules:
     - host: {{ .Values.ingress.hostname | quote }}

diff --git a/charts/superset/values.yaml b/charts/superset/values.yaml
@@ -74,6 +74,8 @@ ingress:
   #  - secretName: chart-example-tls
   #    hosts:
   #      - chart-example.local
+  useCertManager: false
+  certManagerClusterIssuer: ""
 
 userPreferences:
   language: "en"