apache/pulsar-helm-chart
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Missing fix for #152. Bookie Service also needs the prefix on the port name (#172)

Browse Source 
Fixes #158 (This is the second PR - see also https://github.com/apache/pulsar-helm-chart/pull/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`
This commit is contained in:
Frank Kelly 2021-11-09 12:18:26 -05:00 committed by GitHub
parent 5b10f48f5b
commit 617308147d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
charts/pulsar/Chart.yaml
View File

@ -21,7 +21,7 @@ apiVersion: v1
appVersion: "2.7.2" appVersion: "2.7.2"
description: Apache Pulsar Helm chart for Kubernetes description: Apache Pulsar Helm chart for Kubernetes
name: pulsar name: pulsar
version: 2.7.3 version: 2.7.4
home: https://pulsar.apache.org home: https://pulsar.apache.org
sources: sources:
- https://github.com/apache/pulsar - https://github.com/apache/pulsar

2
charts/pulsar/templates/bookkeeper-service.yaml
View File

@ -32,7 +32,7 @@ metadata:
{{- end }} {{- end }}
spec: spec:
ports: ports:
- name: bookie - name: "{{ .Values.tcpPrefix }}bookie"
port: {{ .Values.bookkeeper.ports.bookie }} port: {{ .Values.bookkeeper.ports.bookie }}
- name: http - name: http
port: {{ .Values.bookkeeper.ports.http }} port: {{ .Values.bookkeeper.ports.http }}