[incubator/kafka] Makes updateStrategy fully configurable (#3391)

* Allows kafka RollingUpdate without partitioning

* Fixes linter error

* Bumps kafka chart version

* Simplifies kafka updateStrategy

* Updates kafka nested zookeeper chart

* Bumps kafka chart version
This commit is contained in:
Federico Nusymowicz
2018-01-29 13:19:34 -08:00
committed by k8s-ci-robot
parent a23db4997c
commit 2834ea9e86
6 changed files with 9 additions and 18 deletions
+1 -1
View File
@@ -2,7 +2,7 @@ apiVersion: v1
description: Apache Kafka is publish-subscribe messaging rethought as a distributed
commit log.
name: kafka
version: 0.2.12
version: 0.2.13
keywords:
- kafka
- zookeeper
+1 -2
View File
@@ -65,8 +65,7 @@ following configurable parameters:
| `storage` | Kafka Persistent volume size | `1Gi` |
| `configurationOverrides` | `Kafka ` [configuration setting](https://kafka.apache.org/documentation/#brokerconfigs) overrides in the dictionary format `setting.name: value` | `{}` |
| `schema-registry.enabled` | If True, installs Schema Registry Chart | `false` |
| `updateStrategy` | The type of StatefulSet update strategy to use. One of `OnDelete` or `RollingUpdate` | `OnDelete` |
| `rollingUpdatePartition` | When `RollingUpdate` is the `updateStrategy`, which `partition` of the statefulset is updated first. | `0` |
| `updateStrategy` | StatefulSet update strategy to use. | `{ type: "OnDelete" }` |
| `zookeeper.enabled` | If True, installs Zookeeper Chart | `true` |
| `zookeeper.url` | URL of Zookeeper Cluster (unneeded if installing Zookeeper Chart) | `""` |
| `zookeeper.port` | Port of Zookeeper Cluster | `2181` |
+3 -3
View File
@@ -1,9 +1,9 @@
dependencies:
- name: zookeeper
repository: https://kubernetes-charts-incubator.storage.googleapis.com/
version: 0.4.3
version: 0.4.4
- name: schema-registry
repository: https://kubernetes-charts-incubator.storage.googleapis.com/
version: 0.1.0
digest: sha256:44e2a18583a8ecc8c92ccf042f69f1b656c2e06b01bef6fb03e289b2d511c7fc
generated: 2017-12-23T13:47:40.667268-08:00
digest: sha256:1c6ea22f85337e08488fa9b094e9b740393aabb0d893d5cad3128a0e4cec0e44
generated: 2018-01-27T11:06:29.660226-08:00
+1 -1
View File
@@ -1,6 +1,6 @@
dependencies:
- name: zookeeper
version: 0.4.3
version: 0.4.4
repository: https://kubernetes-charts-incubator.storage.googleapis.com/
condition: zookeeper.enabled
- name: schema-registry
+1 -5
View File
@@ -12,11 +12,7 @@ metadata:
spec:
serviceName: {{ template "kafka.fullname" . }}-headless
updateStrategy:
type: {{ .Values.updateStrategy }}
{{ if eq .Values.updateStrategy "RollingUpdate" -}}
rollingUpdate:
partition: {{ .Values.rollingUpdatePartition }}
{{- end }}
{{ toYaml .Values.updateStrategy | indent 4 }}
replicas: {{ default 3 .Values.replicas }}
template:
metadata:
+2 -6
View File
@@ -31,12 +31,8 @@ storage: "1Gi"
## The StatefulSet Update Strategy which Kafka will use when changes are applied.
## ref: https://kubernetes.io/docs/concepts/workloads/controllers/statefulset/#update-strategies
updateStrategy: "OnDelete"
## If `updateStrategy` is set to "RollingUpdate", you may optionally specify which partition
## should be updated first.
## ref: https://kubernetes.io/docs/concepts/workloads/controllers/statefulset/#partitions
rollingUpdatePartition: 0
updateStrategy:
type: "OnDelete"
## The name of the storage class which the cluster should use.
# storageClass: default