This is an automated email from the ASF dual-hosted git repository.
lhotari 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 bd8bc63 Change default statusFilePath to /pulsar/logs/status (#489)
bd8bc63 is described below
commit bd8bc633dff4843e0ce3404c53b81ac0d0699960
Author: Lari Hotari <[email protected]>
AuthorDate: Mon Apr 15 05:41:17 2024 -0700
Change default statusFilePath to /pulsar/logs/status (#489)
* Change default statusFilePath to /pulsar/logs/status
* Write OK to statusFilePath
---
charts/pulsar/templates/broker-configmap.yaml | 2 +-
charts/pulsar/templates/broker-statefulset.yaml | 2 +-
charts/pulsar/templates/proxy-configmap.yaml | 2 +-
charts/pulsar/templates/proxy-statefulset.yaml | 2 +-
4 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/charts/pulsar/templates/broker-configmap.yaml
b/charts/pulsar/templates/broker-configmap.yaml
index 40f01b8..baac848 100644
--- a/charts/pulsar/templates/broker-configmap.yaml
+++ b/charts/pulsar/templates/broker-configmap.yaml
@@ -41,7 +41,7 @@ data:
exposeTopicLevelMetricsInPrometheus: "true"
numHttpServerThreads: "8"
zooKeeperSessionTimeoutMillis: "30000"
- statusFilePath: "{{ template "pulsar.home" . }}/status"
+ statusFilePath: "{{ template "pulsar.home" . }}/logs/status"
# Tiered storage settings
{{- if .Values.broker.storageOffload.driver }}
diff --git a/charts/pulsar/templates/broker-statefulset.yaml
b/charts/pulsar/templates/broker-statefulset.yaml
index 34938cf..f45adcf 100644
--- a/charts/pulsar/templates/broker-statefulset.yaml
+++ b/charts/pulsar/templates/broker-statefulset.yaml
@@ -225,7 +225,7 @@ spec:
{{- end }}
bin/apply-config-from-env.py conf/broker.conf;
bin/gen-yml-from-env.py conf/functions_worker.yml;
- echo "OK" > status;
+ echo "OK" > "${statusFilePath:-status}";
{{- include "pulsar.broker.zookeeper.tls.settings" . | nindent 10 }}
bin/pulsar zookeeper-shell -server {{ template
"pulsar.zookeeper.connect" . }} get {{ template "pulsar.broker.znode" . }};
while [ $? -eq 0 ]; do
diff --git a/charts/pulsar/templates/proxy-configmap.yaml
b/charts/pulsar/templates/proxy-configmap.yaml
index 5911f01..959c23d 100644
--- a/charts/pulsar/templates/proxy-configmap.yaml
+++ b/charts/pulsar/templates/proxy-configmap.yaml
@@ -28,7 +28,7 @@ metadata:
component: {{ .Values.proxy.component }}
data:
clusterName: {{ template "pulsar.cluster.name" . }}
- statusFilePath: "{{ template "pulsar.home" . }}/status"
+ statusFilePath: "{{ template "pulsar.home" . }}/logs/status"
# prometheus needs to access /metrics endpoint
webServicePort: "{{ .Values.proxy.ports.containerPorts.http }}"
{{- if or (not .Values.tls.enabled) (not .Values.tls.proxy.enabled) }}
diff --git a/charts/pulsar/templates/proxy-statefulset.yaml
b/charts/pulsar/templates/proxy-statefulset.yaml
index 1ea2bfe..e6a1810 100644
--- a/charts/pulsar/templates/proxy-statefulset.yaml
+++ b/charts/pulsar/templates/proxy-statefulset.yaml
@@ -185,7 +185,7 @@ spec:
{{ .Values.proxy.additionalCommand }}
{{- end }}
bin/apply-config-from-env.py conf/proxy.conf &&
- echo "OK" > status &&
+ echo "OK" > "${statusFilePath:-status}" &&
OPTS="${OPTS} -Dlog4j2.formatMsgNoLookups=true" exec bin/pulsar proxy
ports:
# prometheus needs to access /metrics endpoint
