Skip to content

Commit b1540cd

Browse files
authored
Explicitly set pool-namespace flag for wider compatibility (#1633)
1 parent 6e6823a commit b1540cd

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

config/charts/inferencepool/templates/epp-deployment.yaml

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,11 @@ spec:
3333
args:
3434
- --pool-name
3535
- {{ .Release.Name }}
36+
# The pool namespace is optional because EPP can default to the NAMESPACE env var.
37+
# We still keep this here so that the template works with older versions of EPP, or other
38+
# distros of EPP which may not have implemented the NAMESPACE env var defaulting behavior.
39+
- --pool-namespace
40+
- {{ .Release.Namespace }}
3641
{{- if ne .Values.inferencePool.apiVersion "inference.networking.k8s.io" }}
3742
- --pool-group
3843
- "{{ (split "/" .Values.inferencePool.apiVersion)._0 }}"

0 commit comments

Comments
 (0)