From d74d08a89d436f54dce0f5462d9b698cb827f7e8 Mon Sep 17 00:00:00 2001 From: Lari Hotari Date: Mon, 3 Jan 2022 12:59:58 +0200 Subject: [PATCH] Use NIOServerCnxnFactory for Zookeeper to fix NPE issues with Pulsar 2.8.x+ (#180) - follow recommendation in https://github.com/apache/pulsar/issues/11070#issuecomment-936539979 --- charts/pulsar/templates/zookeeper-configmap.yaml | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/charts/pulsar/templates/zookeeper-configmap.yaml b/charts/pulsar/templates/zookeeper-configmap.yaml index 3084dea..6536507 100644 --- a/charts/pulsar/templates/zookeeper-configmap.yaml +++ b/charts/pulsar/templates/zookeeper-configmap.yaml @@ -29,12 +29,15 @@ metadata: component: {{ .Values.zookeeper.component }} data: dataDir: /pulsar/data/zookeeper + {{- if and .Values.tls.enabled .Values.tls.zookeeper.enabled }} + # enable zookeeper tls PULSAR_PREFIX_serverCnxnFactory: org.apache.zookeeper.server.NettyServerCnxnFactory serverCnxnFactory: org.apache.zookeeper.server.NettyServerCnxnFactory - # enable zookeeper tls - {{- if and .Values.tls.enabled .Values.tls.zookeeper.enabled }} secureClientPort: "{{ .Values.zookeeper.ports.clientTls }}" PULSAR_PREFIX_secureClientPort: "{{ .Values.zookeeper.ports.clientTls }}" + {{- else }} + PULSAR_PREFIX_serverCnxnFactory: org.apache.zookeeper.server.NIOServerCnxnFactory + serverCnxnFactory: org.apache.zookeeper.server.NIOServerCnxnFactory {{- end }} {{ toYaml .Values.zookeeper.configData | indent 2 }} {{- end }}