Merge pull request #154 from underpass-ai/feature/vllm-ingress-multi-host

chart: support multiple vLLM ingress hosts

Author: tgarciai

charts/underpass-runtime/templates/ingress-vllm.yaml

@@ -1,4 +1,6 @@
 {{- if .Values.vllm.ingress.enabled }}
+{{- $ingress := .Values.vllm.ingress }}
+{{- $hosts := $ingress.hosts | default (list $ingress.host) }}
 apiVersion: networking.k8s.io/v1
 kind: Ingress
 metadata:
@@ -21,17 +23,21 @@ spec:
   ingressClassName: nginx
   tls:
     - hosts:
-        - {{ .Values.vllm.ingress.host | quote }}
-      secretName: {{ .Values.vllm.ingress.tlsSecret }}
+        {{- range $host := $hosts }}
+        - {{ $host | quote }}
+        {{- end }}
+      secretName: {{ $ingress.tlsSecret }}
   rules:
-    - host: {{ .Values.vllm.ingress.host | quote }}
+    {{- range $host := $hosts }}
+    - host: {{ $host | quote }}
       http:
         paths:
           - path: /
             pathType: Prefix
             backend:
               service:
-                name: {{ .Values.vllm.ingress.service.name }}
+                name: {{ $ingress.service.name }}
                 port:
-                  number: {{ .Values.vllm.ingress.service.port }}
+                  number: {{ $ingress.service.port }}
+    {{- end }}
 {{- end }}

charts/underpass-runtime/values.yaml

@@ -323,8 +323,11 @@ vllm:
     enabled: false
     # -- cert-manager ClusterIssuer used when ingress TLS is enabled
     clusterIssuer: ""
-    # -- Hostname served by the ingress
+    # -- Hostname served by the ingress. Kept for backwards compatibility when
+    # vllm.ingress.hosts is empty.
     host: ""
+    # -- Hostnames served by the ingress
+    hosts: []
     # -- TLS secret name for the ingress certificate
     tlsSecret: ""
     service: