This is an automated email from the ASF dual-hosted git repository.
eolivelli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar-helm-chart.git
The following commit(s) were added to refs/heads/master by this push:
new d74d08a Use NIOServerCnxnFactory for Zookeeper to fix NPE issues with
Pulsar 2.8.x+ (#180)
d74d08a is described below
commit d74d08a89d436f54dce0f5462d9b698cb827f7e8
Author: Lari Hotari <[email protected]>
AuthorDate: Mon Jan 3 12:59:58 2022 +0200
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 }}
