Skip to content

Commit

Permalink
Enabled SSL re-direction in istio gateway (#31)
Browse files Browse the repository at this point in the history
* Updated version to 1.5.2

* corrected kiali chart name in getting port

* Added TLS redirection if TLS is enabled
  • Loading branch information
shehbaz-pathan authored Sep 25, 2024
1 parent aa1bb60 commit 180b42d
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 3 deletions.
2 changes: 1 addition & 1 deletion charts/nopo11y-stack/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -48,4 +48,4 @@ dependencies:
description: A Helm chart for observability stack
name: nopo11y-stack
type: application
version: 1.5.1
version: 1.5.2
2 changes: 1 addition & 1 deletion charts/nopo11y-stack/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -105,7 +105,7 @@
{{- if (index .Values "kiali-server" "enabled") }}
{{- $kiali:= dict }}
{{- $kiali = set $kiali "name" (printf "%s" (include "kiali-server.fullname" (index .Subcharts "kiali-server"))) }}
{{- $kiali = set $kiali "port" .Values.kiali.server.port }}
{{- $kiali = set $kiali "port" (index .Values "kiali-server" "server" "port") }}
{{- if (index .Values "kiali-server" "server" "web_root") }}
{{- if ne (index .Values "kiali-server" "server" "web_root") "/" }}
{{- $kiali = set $kiali "path" (index .Values "kiali-server" "server" "web_root") }}
Expand Down
16 changes: 15 additions & 1 deletion charts/nopo11y-stack/templates/istio-gateway.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ spec:
{{- else }}
number: 80
name: http
protocol: HTTP
protocol: HTTP2
{{- end }}
hosts:
{{- if .Values.nopo11y_ingress.host }}
Expand All @@ -33,5 +33,19 @@ spec:
mode: SIMPLE
credentialName: {{ $.Release.Name }}-{{ $.Release.Namespace }}-tls-secret
{{- end }}
{{- if .Values.nopo11y_ingress.tls.enabled }}
- port:
number: 80
name: http
protocol: HTTP2
hosts:
{{- if .Values.nopo11y_ingress.host }}
- {{ .Values.nopo11y_ingress.host |quote }}
{{- else }}
- "*"
{{- end }}
tls:
httpsRedirect: true
{{- end }}
{{- end }}
{{- end }}

0 comments on commit 180b42d

Please sign in to comment.