Skip to content

Commit

Permalink
Fix: Support GCE in ingress path
Browse files Browse the repository at this point in the history
Compatbility with GKE / GCE, where GCE requires a path that contains /* to function and here we can't use the /* and it messes up the ingress.

NB: let the user choose his own path, if he wants more complicated paths then he will do it on his own after consulting the documentation

Signed-off-by: El Mehdi Bouamama <elmehdi.bouamama@infomineo.com>
  • Loading branch information
ElMehdiBouamama-Infomineo committed Aug 30, 2024
1 parent a12c809 commit 3115907
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions charts/kubernetes-dashboard/templates/networking/ingress.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -62,11 +62,7 @@ spec:
- host: {{ $host }}
http:
paths:
{{- if not (eq $.Values.app.ingress.path "/") }}
- path: {{ $.Values.app.ingress.path }}(/|$)(.*)
{{- else }}
- path: {{ $.Values.app.ingress.path }}
{{- end }}
pathType: {{ $.Values.app.ingress.pathType }}
backend:
service:
Expand Down

0 comments on commit 3115907

Please sign in to comment.