Merge pull request #727 from acend/fix_helmchart_ingress

Use ingressClassName instead on annotation
acend · Oct 24, 2023 · 2a07bc5 · 2a07bc5
2 parents 20c0a05 + 65ad1f9
commit 2a07bc5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/helm-chart/templates/acend-ingres.yaml b/helm-chart/templates/acend-ingres.yaml
@@ -13,6 +13,7 @@ metadata:
     {{- toYaml . | nindent 4 }}
   {{- end }}
 spec:
+  ingressClassName: {{ .Values.ingress.ingressClass }}
   rules:
   - host: acend.ch
     http:
@@ -55,6 +56,7 @@ metadata:
     nginx.ingress.kubernetes.io/auth-secret: staging-basic-auth
   {{- end }}
 spec:
+  ingressClassName: {{ .Values.ingress.ingressClass }}
   rules:
   - host: website-{{ .Release.Name }}.test.acend.ch
     http:

diff --git a/helm-chart/values.yaml b/helm-chart/values.yaml
@@ -11,8 +11,8 @@ image:
   tag: ""
 
 ingress:
+  ingressClass: nginx
   annotations:
-    kubernetes.io/ingress.class: nginx
     nginx.ingress.kubernetes.io/configuration-snippet: |
       more_set_headers "Strict-Transport-Security: max-age=31536000; includeSubDomains";
       more_set_headers "Content-Security-Policy: default-src 'self' https://*.acend.ch";