From 0e23aebd9cd23bda29780e99dfd46a44d3dc5181 Mon Sep 17 00:00:00 2001 From: leomichalski Date: Thu, 19 Mar 2026 16:33:42 -0300 Subject: [PATCH 1/2] feat(helm): allow to specify nodePort of UI Service Signed-off-by: leomichalski --- helm/kagent/templates/ui-service.yaml | 3 +++ helm/kagent/values.yaml | 1 + 2 files changed, 4 insertions(+) diff --git a/helm/kagent/templates/ui-service.yaml b/helm/kagent/templates/ui-service.yaml index 526aa2464..3730e0ac4 100644 --- a/helm/kagent/templates/ui-service.yaml +++ b/helm/kagent/templates/ui-service.yaml @@ -10,6 +10,9 @@ spec: ports: - port: {{ .Values.ui.service.ports.port }} targetPort: {{ .Values.ui.service.ports.targetPort }} + {{ if eq .Values.ui.service.type "NodePort" }} + nodePort: {{ .Values.ui.service.ports.nodePort }} + {{ end }} protocol: TCP name: ui selector: diff --git a/helm/kagent/values.yaml b/helm/kagent/values.yaml index 3726c3702..9fd5f5976 100644 --- a/helm/kagent/values.yaml +++ b/helm/kagent/values.yaml @@ -175,6 +175,7 @@ ui: ports: port: 8080 targetPort: 8080 + nodePort: null env: {} # Additional configuration key-value pairs for the ui ConfigMap # -- Pod-level security context for the UI pod. Overrides the global podSecurityContext. # @default -- (uses global podSecurityContext) From d33c0608bf9a1349b84992d074bcc79e84f098ea Mon Sep 17 00:00:00 2001 From: Leonardo Date: Thu, 19 Mar 2026 17:12:09 -0300 Subject: [PATCH 2/2] fix(helm): invalid yaml when service type is NodePort, but nodePort is null Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Leonardo --- helm/kagent/templates/ui-service.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/helm/kagent/templates/ui-service.yaml b/helm/kagent/templates/ui-service.yaml index 3730e0ac4..b3adee559 100644 --- a/helm/kagent/templates/ui-service.yaml +++ b/helm/kagent/templates/ui-service.yaml @@ -10,9 +10,9 @@ spec: ports: - port: {{ .Values.ui.service.ports.port }} targetPort: {{ .Values.ui.service.ports.targetPort }} - {{ if eq .Values.ui.service.type "NodePort" }} + {{- if and (eq .Values.ui.service.type "NodePort") .Values.ui.service.ports.nodePort }} nodePort: {{ .Values.ui.service.ports.nodePort }} - {{ end }} + {{- end }} protocol: TCP name: ui selector: