Missing fix for datastax#152. Bookie Service also needs the prefix on…

… the port name (datastax#172) Fixes datastax#158 (This is the second PR - see also apache/pulsar-helm-chart#162) ### Motivation * All non-standard port-names need a proper protocol prefix to support Istio https://istio.io/latest/docs/ops/configuration/traffic-management/protocol-selection/#explicit-protocol-selection ### Modifications Add the prefix value before `bookie`
pgier · Nov 9, 2021 · 6173081 · 6173081
1 parent 5b10f48
commit 6173081
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/charts/pulsar/Chart.yaml b/charts/pulsar/Chart.yaml
@@ -21,7 +21,7 @@ apiVersion: v1
 appVersion: "2.7.2"
 description: Apache Pulsar Helm chart for Kubernetes
 name: pulsar
-version: 2.7.3
+version: 2.7.4
 home: https://pulsar.apache.org
 sources:
 - https://github.com/apache/pulsar

diff --git a/charts/pulsar/templates/bookkeeper-service.yaml b/charts/pulsar/templates/bookkeeper-service.yaml
@@ -32,7 +32,7 @@ metadata:
 {{- end }}
 spec:
   ports:
-  - name: bookie
+  - name: "{{ .Values.tcpPrefix }}bookie"
     port: {{ .Values.bookkeeper.ports.bookie }}
   - name: http
     port: {{ .Values.bookkeeper.ports.http }}