Skip to content

Commit b533b29

Browse files
committed
Merge branch 'storage-classes-1.8' into kubernetes-1.8-test
2 parents 8e2de05 + f9c24c5 commit b533b29

File tree

7 files changed

+35
-9
lines changed

7 files changed

+35
-9
lines changed

50kafka.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,7 @@ spec:
7272
name: data
7373
spec:
7474
accessModes: [ "ReadWriteOnce" ]
75+
storageClassName: kafka-broker
7576
resources:
7677
requests:
7778
storage: 200Gi

README.md

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,12 @@ The goal is to provide [Bootstrap servers](http://kafka.apache.org/documentation
3030

3131
Zookeeper at `zookeeper.kafka.svc.cluster.local:2181`.
3232

33+
## Prepare storage classes
34+
35+
For Minikube run `kubectl create -f configure-minikube/`.
36+
37+
There's a similar setup for GKE, `configure/gke-*`. You might want to tweak it before creating.
38+
3339
## Start Zookeeper
3440

3541
The [Kafka book](https://www.confluent.io/resources/kafka-definitive-guide-preview-edition/) recommends that Kafka has its own Zookeeper cluster with at least 5 instances.
@@ -64,14 +70,6 @@ For clusters that enfoce [RBAC](https://kubernetes.io/docs/admin/authorization/r
6470
kubectl apply -f rbac-namespace-default/
6571
```
6672

67-
## Caution: `Delete` Reclaim Policy is default
68-
69-
In production you likely want to [manually set Reclaim Policy](https://kubernetes.io/docs/tasks/administer-cluster/change-pv-reclaim-policy/),
70-
or your data will be gone if the generated [volume claim](https://kubernetes.io/docs/concepts/storage/persistent-volumes/#persistentvolumeclaims)s are deleted.
71-
72-
This can't be done [in manifests](https://github.com/Yolean/kubernetes-kafka/pull/50),
73-
at least not [until Kubernetes 1.8](https://github.com/kubernetes/features/issues/352).
74-
7573
## Tests
7674

7775
Tests are based on the [kube-test](https://github.com/Yolean/kube-test) concept.
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
kind: StorageClass
2+
apiVersion: storage.k8s.io/v1
3+
metadata:
4+
name: kafka-broker
5+
provisioner: k8s.io/minikube-hostpath
Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
kind: StorageClass
2+
apiVersion: storage.k8s.io/v1
3+
metadata:
4+
name: kafka-zookeeper
5+
provisioner: k8s.io/minikube-hostpath
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
kind: StorageClass
2+
apiVersion: storage.k8s.io/v1
3+
metadata:
4+
name: kafka-broker
5+
provisioner: kubernetes.io/gce-pd
6+
reclaimPolicy: Retain
7+
parameters:
8+
type: pd-standard
Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
kind: StorageClass
2+
apiVersion: storage.k8s.io/v1
3+
metadata:
4+
name: kafka-zookeeper
5+
provisioner: kubernetes.io/gce-pd
6+
reclaimPolicy: Retain
7+
parameters:
8+
type: pd-ssd

zookeeper/50pzoo.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,7 @@ spec:
6868
name: data
6969
spec:
7070
accessModes: [ "ReadWriteOnce" ]
71+
storageClassName: kafka-zookeeper
7172
resources:
7273
requests:
73-
storage: 10Gi
74+
storage: 1Gi

0 commit comments

Comments
 (0)