[k8s] make ingress route compatible with multi-instance in the same c…

…luster (#1405)
startreedata · Apr 26, 2024 · e22565f · e22565f
1 parent 05916bc
commit e22565f
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/kubernetes/helm/startree-thirdeye/templates/ui/ingress-route-tls.yaml b/kubernetes/helm/startree-thirdeye/templates/ui/ingress-route-tls.yaml
@@ -23,7 +23,7 @@ spec:
   entryPoints:
   - websecure
   routes:
-  - match: HostSNI(`thirdeye.{{ required "domain is required." .Values.domain }}`)
+  - match: HostSNI(`{{ .Release.Namespace }}.{{ required "domain is required." .Values.domain }}`)
     services:
     - name: {{ include "thirdeye.ui.headless" . }}
       namespace: {{ .Release.Namespace }}

diff --git a/kubernetes/helm/startree-thirdeye/templates/ui/ingress-route-tlsv2.yaml b/kubernetes/helm/startree-thirdeye/templates/ui/ingress-route-tlsv2.yaml
@@ -24,7 +24,7 @@ spec:
   - websecure
   routes:
     - kind: Rule
-      match: Host(`thirdeye.{{ required "domain is required." .Values.domain }}`)
+      match: Host(`{{ .Release.Namespace }}.{{ required "domain is required." .Values.domain }}`)
       services:
       - name: {{ include "thirdeye.ui.headless" . }}
         namespace: {{ .Release.Namespace }}

diff --git a/kubernetes/helm/startree-thirdeye/templates/ui/ingress-route.yaml b/kubernetes/helm/startree-thirdeye/templates/ui/ingress-route.yaml
@@ -36,10 +36,10 @@ spec:
   entryPoints:
     - web
   routes:
-    - match: Host(`thirdeye.{{ required "domain is required." .Values.domain }}`)
+    - match: Host(`{{ .Release.Namespace }}.{{ required "domain is required." .Values.domain }}`)
       kind: Rule
       services:
         - name: {{ include "thirdeye.ui.headless" . }}
           namespace: {{ .Release.Namespace }}
           port: {{ .Values.ui.port }}
-{{- end }}
+{{- end }}