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

nwang 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 135fb76  Helm chart Bookkeeper fix (#3381)
135fb76 is described below

commit 135fb760b00739dbb0f655fb91937a4edd0cb8a0
Author: Nicholas Nezis <[email protected]>
AuthorDate: Thu Nov 7 03:11:16 2019 -0500

    Helm chart Bookkeeper fix (#3381)
    
    * Kubernetes Statefulset bug fix
    
    * Update bookie.yaml
    
    Updated to only use hostname with Statefulset
---
 deploy/kubernetes/helm/templates/bookie.yaml | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/deploy/kubernetes/helm/templates/bookie.yaml 
b/deploy/kubernetes/helm/templates/bookie.yaml
index fecc7f1..acb359e 100644
--- a/deploy/kubernetes/helm/templates/bookie.yaml
+++ b/deploy/kubernetes/helm/templates/bookie.yaml
@@ -41,6 +41,9 @@ data:
   BK_indexDirectories: "/bookkeeper/data/ledgers" 
   BK_zkServers: {{ .Release.Name }}-zookeeper:{{ .Values.zookeeper.clientPort 
}}
   BK_autoRecoveryDaemonEnabled: "true"
+  {{- if or (eq .Values.platform "gke") (eq .Values.platform "minikube") }}
+  BK_useHostNameAsBookieID: "true"
+  {{- end }}
   # TODO: Issue 458: https://github.com/apache/bookkeeper/issues/458
   {{- if eq .Values.bookkeeper.prometheus.enabled true }}
   BK_enableStatistics: "true"

Reply via email to