Skip to content

Commit

Permalink
Fix ingress
Browse files Browse the repository at this point in the history
  • Loading branch information
thisisnotashwin committed Jul 28, 2023
1 parent 3e1f799 commit 0bf7a62
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
3 changes: 3 additions & 0 deletions .changelog/2687.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
```release-note:bug
helm: fix ui ingress manifest formatting, and exclude `ingressClass` when not defined.
```
4 changes: 3 additions & 1 deletion charts/consul/templates/ui-ingress.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,9 +25,11 @@ metadata:
{{ tpl .Values.ui.ingress.annotations . | nindent 4 | trim }}
{{- end }}
spec:
{{- if ne .Values.ui.ingress.ingressClassName "" }}
ingressClassName: {{ .Values.ui.ingress.ingressClassName }}
{{- end }}
rules:
{{ $global := .Values.global }}
{{- $global := .Values.global }}
{{- if or ( gt .Capabilities.KubeVersion.Major "1" ) ( ge .Capabilities.KubeVersion.Minor "19" ) }}
{{- range .Values.ui.ingress.hosts }}
- host: {{ .host | quote }}
Expand Down

0 comments on commit 0bf7a62

Please sign in to comment.