Skip to content

Commit c51ba21

Browse files
committed
Shorter, possibly clearer
1 parent b3ac6b7 commit c51ba21

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

README.md

+5-5
Original file line numberDiff line numberDiff line change
@@ -52,13 +52,13 @@ For clients we tend to use [librdkafka](https://github.com/edenhill/librdkafka)-
5252
To use [Kafka Connect](http://kafka.apache.org/documentation/#connect) and [Kafka Streams](http://kafka.apache.org/documentation/streams/) you may want to take a look at our [sample](https://github.com/solsson/dockerfiles/tree/master/connect-files) [Dockerfile](https://github.com/solsson/dockerfiles/tree/master/streams-logfilter)s.
5353
Don't forget the [addon](https://github.com/Yolean/kubernetes-kafka/labels/addon)s.
5454

55-
## Set "reclaim policy" for persistent volumes
55+
## Caution: `Delete` Reclaim Policy is default
5656

57-
Caution: For each new kafka and pzoo pod you have to [manually set Reclaim Policy](https://kubernetes.io/docs/tasks/administer-cluster/change-pv-reclaim-policy/)
58-
to avoid losing your data if the [PVC](https://kubernetes.io/docs/concepts/storage/persistent-volumes/#persistentvolumeclaims)s or [PV](https://kubernetes.io/docs/concepts/storage/persistent-volumes/#persistent-volumes)s are deleted,
59-
including if the `kafka` namespace is deleted and recreated.
57+
In production you likely want to [manually set Reclaim Policy](https://kubernetes.io/docs/tasks/administer-cluster/change-pv-reclaim-policy/),
58+
our your data will be gone if the generated [volume claim](https://kubernetes.io/docs/concepts/storage/persistent-volumes/#persistentvolumeclaims)s are deleted.
6059

61-
See [feature#352](https://github.com/kubernetes/features/issues/352) [#38192](https://github.com/kubernetes/kubernetes/issues/38192) [#47987](https://github.com/kubernetes/kubernetes/pull/47987) for details on why this isn't supported (yet) in manifests.
60+
This can't be done [in manifests](https://github.com/Yolean/kubernetes-kafka/pull/50),
61+
at least not [until Kubernetes 1.8](https://github.com/kubernetes/features/issues/352).
6262

6363
# Tests
6464

0 commit comments

Comments
 (0)