This is an automated email from the ASF dual-hosted git repository.

wankai pushed a commit to branch update-banyandb
in repository https://gitbox.apache.org/repos/asf/skywalking-helm.git


The following commit(s) were added to refs/heads/update-banyandb by this push:
     new a21d2a7  fix targets
a21d2a7 is described below

commit a21d2a7675c29e925d99528bb2aadfde22effbeb
Author: wankai123 <[email protected]>
AuthorDate: Tue Nov 14 21:36:54 2023 +0800

    fix targets
---
 chart/skywalking/templates/_helpers.tpl | 6 +-----
 chart/skywalking/values.yaml            | 3 +--
 2 files changed, 2 insertions(+), 7 deletions(-)

diff --git a/chart/skywalking/templates/_helpers.tpl 
b/chart/skywalking/templates/_helpers.tpl
index dc4740d..87631c1 100644
--- a/chart/skywalking/templates/_helpers.tpl
+++ b/chart/skywalking/templates/_helpers.tpl
@@ -160,11 +160,7 @@ Create the name of the service account to use for the 
satellite cluster
 - name: SW_DATA_SOURCE_PASSWORD
   value: "{{ .Values.postgresql.auth.password }}"
 {{- else if eq .Values.oap.storageType "banyandb" }}
-{{- $banyandbHost := "banyandb-grpc" -}}
-{{- if not .Values.banyandb.enabled -}}
-{{- $banyandbHost = .Values.banyandb.config.grpcHost -}}
-{{- end }}
 - name: SW_STORAGE_BANYANDB_TARGETS
-  value: "{{ $banyandbHost }}:{{ .Values.banyandb.config.grpcPort }}"
+  value: "{{ .Values.banyandb.config.targets }}"
 {{- end }}
 {{- end -}}
diff --git a/chart/skywalking/values.yaml b/chart/skywalking/values.yaml
index 5c323eb..67d1975 100644
--- a/chart/skywalking/values.yaml
+++ b/chart/skywalking/values.yaml
@@ -449,10 +449,9 @@ postgresql:
 banyandb:
   enabled: false
   config:
-    grpcHost: banyandb-grpc
     httpHost: banyandb-http
-    grpcPort: 17912
     httpPort: 17913
+    targets: "banyandb-grpc:17913"
 
 satellite:
   name: satellite

Reply via email to