Skip to content

Commit

Permalink
Update banyandb client connection (#140)
Browse files Browse the repository at this point in the history
  • Loading branch information
wankai123 authored Nov 14, 2023
1 parent e32d22e commit e651c38
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 12 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/e2e.ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,9 +43,9 @@ jobs:
config: test/e2e/swck/e2e.yaml
name: ${{ matrix.test.name }}
env:
OAP_TAG: 14fbd975be077082ed6fa604e05479dd9af0d995
OAP_TAG: 965dea898986ed2ab1343559fb20d6faed11ada9
OAP_REPO: ghcr.io/apache/skywalking/oap
UI_TAG: 14fbd975be077082ed6fa604e05479dd9af0d995
UI_TAG: 965dea898986ed2ab1343559fb20d6faed11ada9
UI_REPO: ghcr.io/apache/skywalking/ui
SATELLITE_TAG: v35bfaff6352b4dc351a706772796a1f79b651c14
SATELLITE_REPO: ghcr.io/apache/skywalking-satellite/skywalking-satellite
Expand Down
10 changes: 2 additions & 8 deletions chart/skywalking/templates/_helpers.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -160,13 +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_HOST
value: "{{ $banyandbHost }}"
- name: SW_STORAGE_BANYANDB_PORT
value: "{{ .Values.banyandb.config.grpcPort }}"
- name: SW_STORAGE_BANYANDB_TARGETS
value: "{{ .Values.banyandb.config.targets }}"
{{- end }}
{{- end -}}
3 changes: 1 addition & 2 deletions chart/skywalking/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -449,10 +449,9 @@ postgresql:
banyandb:
enabled: false
config:
grpcHost: banyandb-grpc
httpHost: banyandb-http
grpcPort: 17912
httpPort: 17913
targets: "banyandb-grpc:17912"

satellite:
name: satellite
Expand Down

0 comments on commit e651c38

Please sign in to comment.