This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pulsar.git
The following commit(s) were added to refs/heads/master by this push: new ac97846 Add bookkeeper service and other fixes (#3026) ac97846 is described below commit ac978462413a32a9b533c250a424cb52be110059 Author: Byron Ruth <b...@devel.io> AuthorDate: Wed Nov 21 16:51:21 2018 -0500 Add bookkeeper service and other fixes (#3026) * Add bookkeeper service and other fixes - Add the topologyKey for the pod affinity for bookies - Fixed a misspelling in the name of the ledgers disk - Fixed the selector in the proxy service to select proxy pods Signed-off-by: Byron Ruth <b...@devel.io> * Update bookkeeper service comment to note DNS Signed-off-by: Byron Ruth <b...@devel.io> --- .../google-kubernetes-engine/bookie.yaml | 32 +++++++++++++++++++--- .../kubernetes/google-kubernetes-engine/proxy.yaml | 2 +- 2 files changed, 29 insertions(+), 5 deletions(-) diff --git a/deployment/kubernetes/google-kubernetes-engine/bookie.yaml b/deployment/kubernetes/google-kubernetes-engine/bookie.yaml index e5caf7c..e08cefd 100644 --- a/deployment/kubernetes/google-kubernetes-engine/bookie.yaml +++ b/deployment/kubernetes/google-kubernetes-engine/bookie.yaml @@ -27,7 +27,7 @@ metadata: provisioner: kubernetes.io/gce-pd parameters: type: pd-ssd - zones: us-central1-a + zones: us-east1-b --- # HDDs for bookie ledger storage apiVersion: storage.k8s.io/v1 @@ -37,7 +37,7 @@ metadata: provisioner: kubernetes.io/gce-pd parameters: type: pd-standard - zones: us-central1-a + zones: us-east1-b --- apiVersion: v1 kind: ConfigMap @@ -67,7 +67,7 @@ spec: labels: app: pulsar component: bookkeeper - cluster: pulsar-gke + cluster: us-east annotations: prometheus.io/scrape: "true" prometheus.io/port: "8000" @@ -78,6 +78,7 @@ spec: preferredDuringSchedulingIgnoredDuringExecution: - weight: 1 podAffinityTerm: + topologyKey: "kubernetes.io/hostname" labelSelector: matchExpressions: - key: "app" @@ -143,7 +144,7 @@ spec: storage: 10Gi storageClassName: bookie-ssd - metadata: - name: ledger-disk + name: ledgers-disk annotations: volume.alpha.kubernetes.io/storage-class: default labels: @@ -155,6 +156,29 @@ spec: storage: 10Gi storageClassName: bookie-hdd + +--- +## +## Bookie headless service. +## This is required for resolving intra-cluser hostnames (via k8s DNS) +## from the brokers. +## +apiVersion: v1 +kind: Service +metadata: + name: bookkeeper + labels: + app: pulsar + component: bookkeeper +spec: + ports: + - port: 3181 + name: server + clusterIP: None + selector: + app: pulsar + component: bookkeeper + --- ## ## Run BookKeeper auto-recovery from a different set of containers diff --git a/deployment/kubernetes/google-kubernetes-engine/proxy.yaml b/deployment/kubernetes/google-kubernetes-engine/proxy.yaml index 138f12f..9f3c25c 100644 --- a/deployment/kubernetes/google-kubernetes-engine/proxy.yaml +++ b/deployment/kubernetes/google-kubernetes-engine/proxy.yaml @@ -81,4 +81,4 @@ spec: protocol: TCP selector: app: pulsar - component: broker + component: proxy