Merge pull request #1 from unikorn-cloud/optional_class

Make Ingress Classes Optional
unikorn-cloud · Mar 13, 2024 · a791e99 · a791e99
2 parents 296485f + 96b0d3f
commit a791e99
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 5 deletions.
diff --git a/charts/cert-manager-issuers/Chart.yaml b/charts/cert-manager-issuers/Chart.yaml
@@ -1,6 +1,6 @@
 apiVersion: v2
-version: v1.0.0
+version: v1.0.1
 name: cert-manager-issuers
 description: A Helm chart to deploy default cert-manager issuers.
 type: application
-icon: https://raw.githubusercontent.com/eschercloudai/helm-addons/main/icons/default.png
+icon: https://raw.githubusercontent.com/unikorn-cloud/unikorn/main/icons/default.png
diff --git a/charts/cert-manager-issuers/templates/clusterissuer-production.yaml b/charts/cert-manager-issuers/templates/clusterissuer-production.yaml
@@ -10,5 +10,7 @@ spec:
     solvers:
     - http01:
         ingress:
-          class: {{ .Values.ingressClass }}
+          {{- with $class := .Values.ingressClass }}
+          ingressClassName: {{ $class }}
+          {{- end }}
           serviceType: {{ .Values.serviceType }}
diff --git a/charts/cert-manager-issuers/templates/clusterissuer-staging.yaml b/charts/cert-manager-issuers/templates/clusterissuer-staging.yaml
@@ -10,5 +10,7 @@ spec:
     solvers:
     - http01:
         ingress:
-          class: {{ .Values.ingressClass }}
+          {{- with $class := .Values.ingressClass }}
+          ingressClassName: {{ $class }}
+          {{- end }}
           serviceType: {{ .Values.serviceType }}
diff --git a/charts/cert-manager-issuers/values.yaml b/charts/cert-manager-issuers/values.yaml
@@ -1,5 +1,5 @@
 # Sets the ingress class of all issuers.
-ingressClass: nginx
+ingressClass: ~
 
 # serviceType defines the service type to use, may be NodePort or ClusterIP
 serviceType: ClusterIP