From 7abfc3a38f02342dc09675547e4ba52bd600b7a6 Mon Sep 17 00:00:00 2001 From: zymap Date: Tue, 6 Feb 2024 21:40:20 +0800 Subject: [PATCH 1/4] Make the backup can run when zookeeper reconfig enabled --- .../templates/zookeeper/zookeeper-backup-configmap.yaml | 4 ---- .../templates/zookeeper/zookeeper-backup-statefulset.yaml | 2 +- 2 files changed, 1 insertion(+), 5 deletions(-) diff --git a/charts/sn-platform/templates/zookeeper/zookeeper-backup-configmap.yaml b/charts/sn-platform/templates/zookeeper/zookeeper-backup-configmap.yaml index 7ef90765..70b8770a 100644 --- a/charts/sn-platform/templates/zookeeper/zookeeper-backup-configmap.yaml +++ b/charts/sn-platform/templates/zookeeper/zookeeper-backup-configmap.yaml @@ -48,10 +48,6 @@ data: secureClientPort: "{{ .Values.zookeeper.ports.clientTls }}" PULSAR_PREFIX_secureClientPort: "{{ .Values.zookeeper.ports.clientTls }}" {{- end }} - {{- if .Values.zookeeper.reconfig.enabled }} - PULSAR_PREFIX_reconfigEnabled: "true" - PULSAR_PREFIX_quorumListenOnAllIPs: "true" - {{- end }} PULSAR_PREFIX_peerType: "observer" {{ toYaml .Values.zookeeper.customTools.backup.configData | indent 2 }} # Include log configuration file, If you want to configure the log level and other configuration diff --git a/charts/sn-platform/templates/zookeeper/zookeeper-backup-statefulset.yaml b/charts/sn-platform/templates/zookeeper/zookeeper-backup-statefulset.yaml index ede25c65..74adf966 100644 --- a/charts/sn-platform/templates/zookeeper/zookeeper-backup-statefulset.yaml +++ b/charts/sn-platform/templates/zookeeper/zookeeper-backup-statefulset.yaml @@ -154,7 +154,7 @@ spec: {{ range $i, $e := until (.Values.zookeeper.replicaCount | int) }}{{ if ne $i 0 }},{{ end }}{{ template "pulsar.fullname" $global }}-{{ $global.Values.zookeeper.component }}-{{ printf "%d" $i }}{{ end }} envFrom: - configMapRef: - name: "{{ template "pulsar.fullname" . }}-{{ .Values.zookeeper.component }}" + name: "{{ template "pulsar.fullname" . }}-backup" {{- if .Values.zookeeper.probe.readiness.enabled }} readinessProbe: exec: From 81cd1d12568c58eefab140662f83599aaeaae3c2 Mon Sep 17 00:00:00 2001 From: zymap Date: Thu, 7 Mar 2024 15:06:16 +0800 Subject: [PATCH 2/4] Apply change to the platform slim --- .../templates/zookeeper/zookeeper-backup-configmap.yaml | 4 ---- .../templates/zookeeper/zookeeper-backup-statefulset.yaml | 2 +- 2 files changed, 1 insertion(+), 5 deletions(-) diff --git a/charts/sn-platform-slim/templates/zookeeper/zookeeper-backup-configmap.yaml b/charts/sn-platform-slim/templates/zookeeper/zookeeper-backup-configmap.yaml index 7ef90765..70b8770a 100644 --- a/charts/sn-platform-slim/templates/zookeeper/zookeeper-backup-configmap.yaml +++ b/charts/sn-platform-slim/templates/zookeeper/zookeeper-backup-configmap.yaml @@ -48,10 +48,6 @@ data: secureClientPort: "{{ .Values.zookeeper.ports.clientTls }}" PULSAR_PREFIX_secureClientPort: "{{ .Values.zookeeper.ports.clientTls }}" {{- end }} - {{- if .Values.zookeeper.reconfig.enabled }} - PULSAR_PREFIX_reconfigEnabled: "true" - PULSAR_PREFIX_quorumListenOnAllIPs: "true" - {{- end }} PULSAR_PREFIX_peerType: "observer" {{ toYaml .Values.zookeeper.customTools.backup.configData | indent 2 }} # Include log configuration file, If you want to configure the log level and other configuration diff --git a/charts/sn-platform-slim/templates/zookeeper/zookeeper-backup-statefulset.yaml b/charts/sn-platform-slim/templates/zookeeper/zookeeper-backup-statefulset.yaml index bcbc7c64..6aea65bb 100644 --- a/charts/sn-platform-slim/templates/zookeeper/zookeeper-backup-statefulset.yaml +++ b/charts/sn-platform-slim/templates/zookeeper/zookeeper-backup-statefulset.yaml @@ -154,7 +154,7 @@ spec: {{ range $i, $e := until (.Values.zookeeper.replicaCount | int) }}{{ if ne $i 0 }},{{ end }}{{ template "pulsar.fullname" $global }}-{{ $global.Values.zookeeper.component }}-{{ printf "%d" $i }}{{ end }} envFrom: - configMapRef: - name: "{{ template "pulsar.fullname" . }}-{{ .Values.zookeeper.component }}" + name: "{{ template "pulsar.fullname" . }}-backup" {{- if .Values.zookeeper.probe.readiness.enabled }} readinessProbe: exec: From 77266cf4751b09e8ef971bb824e8db149c07d898 Mon Sep 17 00:00:00 2001 From: ericsyh Date: Thu, 7 Mar 2024 15:28:21 +0800 Subject: [PATCH 3/4] fix chart lint Signed-off-by: ericsyh --- charts/sn-platform-slim/Chart.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/charts/sn-platform-slim/Chart.yaml b/charts/sn-platform-slim/Chart.yaml index 9770bdbd..c30e3c69 100644 --- a/charts/sn-platform-slim/Chart.yaml +++ b/charts/sn-platform-slim/Chart.yaml @@ -28,4 +28,4 @@ sources: icon: https://raw.githubusercontent.com/streamnative/charts/master/static/logo.svg maintainers: - name: StreamNative Support - email: support@streamnative.io \ No newline at end of file + email: support@streamnative.io From 0ccebb34994cefb7b230a4cf2205a0c8f461d5d1 Mon Sep 17 00:00:00 2001 From: ericsyh Date: Thu, 7 Mar 2024 15:34:01 +0800 Subject: [PATCH 4/4] fix chart lint Signed-off-by: ericsyh --- charts/sn-platform-slim/values.yaml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/charts/sn-platform-slim/values.yaml b/charts/sn-platform-slim/values.yaml index a42de041..cd29504e 100644 --- a/charts/sn-platform-slim/values.yaml +++ b/charts/sn-platform-slim/values.yaml @@ -1042,7 +1042,7 @@ broker: format: "text" # The template field can totally change the log config of the component. The value is a string, which is the content of the log config file. template: {} - + # use a component name that matches your grafana configuration # so the metrics are correctly rendered in grafana dashboard component: broker @@ -1202,8 +1202,8 @@ broker: delayedDeliveryTrackerFactoryClassName: "org.apache.pulsar.broker.delayed.BucketDelayedDeliveryTrackerFactory" # Enable transaction buffer segmented snapshot transactionBufferSegmentedSnapshotEnabled: "true" - # Enable the new Load Balancer, please make sure your cluster is already on Pulsar 3.0 then to use. - # Mix of new and previous Load Balancer in the same cluster will lead to unexpected problems. + # Enable the new Load Balancer, please make sure your cluster is already on Pulsar 3.0 then to use. + # Mix of new and previous Load Balancer in the same cluster will lead to unexpected problems. # loadManagerClassName: "org.apache.pulsar.broker.loadbalance.extensions.ExtensibleLoadManagerImpl" # loadBalancerLoadSheddingStrategy: "org.apache.pulsar.broker.loadbalance.extensions.scheduler.TransferShedder" ### Broker service account