From 0f863124dea00a3edd814f18684cbbec13ed70c1 Mon Sep 17 00:00:00 2001
From: Your Name <conliu@google.com>
Date: Mon, 22 Sep 2025 14:20:57 -0700
Subject: [PATCH] Explicitly set pool-namespace flag for wider compatibility

---
 config/charts/inferencepool/templates/epp-deployment.yaml | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/config/charts/inferencepool/templates/epp-deployment.yaml b/config/charts/inferencepool/templates/epp-deployment.yaml
index 6ed60b20d..39f74f194 100644
--- a/config/charts/inferencepool/templates/epp-deployment.yaml
+++ b/config/charts/inferencepool/templates/epp-deployment.yaml
@@ -33,6 +33,11 @@ spec:
         args:
         - --pool-name
         - {{ .Release.Name }}
+        # The pool namespace is optional because EPP can default to the NAMESPACE env var.
+        # We still keep this here so that the template works with older versions of EPP, or other
+        # distros of EPP which may not have implemented the NAMESPACE env var defaulting behavior.
+        - --pool-namespace
+        - {{ .Release.Namespace }}
         {{- if ne .Values.inferencePool.apiVersion "inference.networking.k8s.io" }}
         - --pool-group
         - "{{ (split "/" .Values.inferencePool.apiVersion)._0 }}"