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
