From 8a0782c112646913dbdabb47c4cb4846bf70eb14 Mon Sep 17 00:00:00 2001 From: Yusuf Daud Date: Wed, 1 Jul 2026 15:14:31 +0100 Subject: [PATCH 1/2] fix: omit empty broker clusterIP to avoid GitOps drift Omit clusterIP from the broker Service manifest when the value is empty, matching standalone-service.yaml. Prevents perpetual Argo CD OutOfSync when Kubernetes assigns a ClusterIP after apply. Fixes apache/pulsar-helm-chart#700 --- charts/pulsar/templates/broker-service.yaml | 2 ++ 1 file changed, 2 insertions(+) diff --git a/charts/pulsar/templates/broker-service.yaml b/charts/pulsar/templates/broker-service.yaml index a761af6c..783815c5 100644 --- a/charts/pulsar/templates/broker-service.yaml +++ b/charts/pulsar/templates/broker-service.yaml @@ -46,7 +46,9 @@ spec: - name: "{{ .Values.tlsPrefix }}pulsarssl" port: {{ .Values.broker.ports.pulsarssl }} {{- end }} + {{- if ne .Values.broker.service.clusterIP "" }} clusterIP: "{{ .Values.broker.service.clusterIP }}" + {{- end }} selector: {{- include "pulsar.matchLabels" . | nindent 4 }} component: {{ .Values.broker.component }} From 4af02df81caf76dc22b19b1b8c1f18be1890f64d Mon Sep 17 00:00:00 2001 From: Lari Hotari Date: Tue, 7 Jul 2026 13:27:35 +0300 Subject: [PATCH 2/2] use a truly check --- charts/pulsar/templates/broker-service.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/charts/pulsar/templates/broker-service.yaml b/charts/pulsar/templates/broker-service.yaml index 783815c5..d02d64fb 100644 --- a/charts/pulsar/templates/broker-service.yaml +++ b/charts/pulsar/templates/broker-service.yaml @@ -46,7 +46,7 @@ spec: - name: "{{ .Values.tlsPrefix }}pulsarssl" port: {{ .Values.broker.ports.pulsarssl }} {{- end }} - {{- if ne .Values.broker.service.clusterIP "" }} + {{- if .Values.broker.service.clusterIP }} clusterIP: "{{ .Values.broker.service.clusterIP }}" {{- end }} selector: