Renaming includeClassAnno to useOldClassField

webrecorder · Jan 9, 2025 · 9b70c77 · 9b70c77
1 parent 1fabb79
commit 9b70c77
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/chart/templates/ingress.yaml b/chart/templates/ingress.yaml
@@ -102,7 +102,7 @@ spec:
     - http01:
         ingress:
           ingressClassName: {{ .Values.ingress_class | default "nginx" }}
-          {{- if .Values.ingress.includeClassAnno }}
+          {{- if .Values.ingress.useOldClassField }}
           class: {{ .Values.ingress_class | default "nginx" }}
           {{- end }}
 

diff --git a/chart/values.yaml b/chart/values.yaml
@@ -450,7 +450,7 @@ ingress:
   # This is related to the issues webrecorder/browsertrix#1570 and cert-manager/cert-manager#6184
   # This relates to the new `http01.ingress.ingressClassName` solver spec attribute which got introduced in the recent v1.15 cert-manager version.
   # Cert-manager v1.15 does not want the fields ingressClassName and class be set at the same time.
-  includeClassAnno: false
+  useOldClassField: false
 
   # Optional: Uncomment to use your own cluster-issuer instead of default ACME https validation
   # custom_cluster_issuer: custom_cluster_issuer-name