Skip to content

Commit

Permalink
remove default inress annotation so the ingressClassName is setable
Browse files Browse the repository at this point in the history
Signed-off-by: Richard Lavey <[email protected]>
  • Loading branch information
rlaveycal committed Apr 27, 2023
1 parent 180aae1 commit eb0f4bb
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
2 changes: 2 additions & 0 deletions chart/openfaas/templates/ingress.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,12 @@ metadata:
chart: {{ .Chart.Name }}-{{ .Chart.Version | replace "+" "_" }}
release: {{ .Release.Name }}
heritage: {{ .Release.Service }}
{{- if .Values.ingress.annotations }}
annotations:
{{- range $key, $value := .Values.ingress.annotations }}
{{ $key }}: {{ $value | quote }}
{{- end }}
{{- end }}
spec:
{{- if .Values.ingress.ingressClassName }}
ingressClassName: {{ .Values.ingress.ingressClassName }}
Expand Down
3 changes: 1 addition & 2 deletions chart/openfaas/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -291,8 +291,7 @@ ingress:
serviceName: gateway
servicePort: 8080
path: /
annotations:
kubernetes.io/ingress.class: nginx
annotations: {}
tls:
# Secrets must be manually created in the namespace.

Expand Down

0 comments on commit eb0f4bb

Please sign in to comment.