[incubator-heron] branch master updated: Removing unused Bookkeeper hostPort that causes port conflict (#3677)

[ni...@apache.org]

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

nicknezis pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-heron.git


The following commit(s) were added to refs/heads/master by this push:
     new d4d3ccc  Removing unused Bookkeeper hostPort that causes port conflict (#3677)
d4d3ccc is described below

commit d4d3cccf32a3a108623c62cb970a3d0b4c5746c3
Author: Nicholas Nezis <ni...@gmail.com>
AuthorDate: Thu Feb 18 22:05:05 2021 -0500

    Removing unused Bookkeeper hostPort that causes port conflict (#3677)
---
 deploy/kubernetes/helm/templates/bookie.yaml | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/deploy/kubernetes/helm/templates/bookie.yaml b/deploy/kubernetes/helm/templates/bookie.yaml
index 12e174f..8ed27ed 100644
--- a/deploy/kubernetes/helm/templates/bookie.yaml
+++ b/deploy/kubernetes/helm/templates/bookie.yaml
@@ -160,9 +160,6 @@ spec:
           ports:
             - name: client
               containerPort: 3181
-              # we are using `status.hostIP` for the bookie's advertised address. export 3181 as the hostPort,
-              # so that the containers are able to access the host port
-              hostPort: 3181
           envFrom:
             - configMapRef:
                 name: {{ .Release.Name }}-bookie-config