[bitnami/kafka] PDB review (#25938)

* [bitnami/kafka] PDB review

Signed-off-by: Fran Mulero <fmulero@vmware.com>

* Update README.md with readme-generator-for-helm

Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>

* Update CHANGELOG.md

Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>

* Update CHANGELOG.md

Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>

* Update README.md with readme-generator-for-helm

Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>

* Update CHANGELOG.md

Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>

* Update CHANGELOG.md

Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>

* Remove metrics PDB

Signed-off-by: Fran Mulero <fmulero@vmware.com>

* Bump patch version

Signed-off-by: Fran Mulero <fmulero@vmware.com>

* Update README.md with readme-generator-for-helm

Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>

---------

Signed-off-by: Fran Mulero <fmulero@vmware.com>
Signed-off-by: Carlos Rodríguez Hernández <carlosrh@vmware.com>
Signed-off-by: Bitnami Containers <bitnami-bot@vmware.com>
Co-authored-by: Carlos Rodríguez Hernández <carlosrh@vmware.com>
Co-authored-by: Bitnami Containers <bitnami-bot@vmware.com>
This commit is contained in:
Fran Mulero
2024-05-29 12:44:07 +02:00
committed by GitHub
parent 2cf771d70c
commit 568aafa1de
8 changed files with 28 additions and 21 deletions

View File

@@ -1,8 +1,12 @@
# Changelog
## 29.1.1 (2024-05-27)
## 29.1.2 (2024-05-29)
* [bitnami/kafka] Fixed Network-Policies for jmx metrics export ([#26369](https://github.com/bitnami/charts/pull/26369))
* [bitnami/kafka] PDB review ([#25938](https://github.com/bitnami/charts/pull/25938))
## <small>29.1.1 (2024-05-28)</small>
* [bitnami/kafka] Fixed Network-Policies for jmx metrics export (#26369) ([7f9445f](https://github.com/bitnami/charts/commit/7f9445f21003a77971d1e7d9c31cf952b6ba9554)), closes [#26369](https://github.com/bitnami/charts/issues/26369)
## 29.1.0 (2024-05-27)

View File

@@ -40,4 +40,4 @@ maintainers:
name: kafka
sources:
- https://github.com/bitnami/charts/tree/main/bitnami/kafka
version: 29.1.1
version: 29.1.2

View File

@@ -644,9 +644,9 @@ You can enable this initContainer by setting `volumePermissions.enabled` to `tru
| `controller.autoscaling.hpa.maxReplicas` | Maximum number of Kafka Controller replicas | `""` |
| `controller.autoscaling.hpa.targetCPU` | Target CPU utilization percentage | `""` |
| `controller.autoscaling.hpa.targetMemory` | Target Memory utilization percentage | `""` |
| `controller.pdb.create` | Deploy a pdb object for the Kafka pod | `false` |
| `controller.pdb.create` | Deploy a pdb object for the Kafka pod | `true` |
| `controller.pdb.minAvailable` | Maximum number/percentage of unavailable Kafka replicas | `""` |
| `controller.pdb.maxUnavailable` | Maximum number/percentage of unavailable Kafka replicas | `1` |
| `controller.pdb.maxUnavailable` | Maximum number/percentage of unavailable Kafka replicas | `""` |
| `controller.persistence.enabled` | Enable Kafka data persistence using PVC, note that ZooKeeper persistence is unaffected | `true` |
| `controller.persistence.existingClaim` | A manually managed Persistent Volume and Claim | `""` |
| `controller.persistence.storageClass` | PVC Storage Class for Kafka data volume | `""` |
@@ -751,9 +751,9 @@ You can enable this initContainer by setting `volumePermissions.enabled` to `tru
| `broker.extraVolumeMounts` | Optionally specify extra list of additional volumeMounts for the Kafka container(s) | `[]` |
| `broker.sidecars` | Add additional sidecar containers to the Kafka pod(s) | `[]` |
| `broker.initContainers` | Add additional Add init containers to the Kafka pod(s) | `[]` |
| `broker.pdb.create` | Deploy a pdb object for the Kafka pod | `false` |
| `broker.pdb.create` | Deploy a pdb object for the Kafka pod | `true` |
| `broker.pdb.minAvailable` | Maximum number/percentage of unavailable Kafka replicas | `""` |
| `broker.pdb.maxUnavailable` | Maximum number/percentage of unavailable Kafka replicas | `1` |
| `broker.pdb.maxUnavailable` | Maximum number/percentage of unavailable Kafka replicas | `""` |
### Experimental: Kafka Broker Autoscaling configuration

View File

@@ -3,8 +3,7 @@ Copyright Broadcom, Inc. All Rights Reserved.
SPDX-License-Identifier: APACHE-2.0
*/}}
{{- $replicaCount := int .Values.broker.replicaCount }}
{{- if and .Values.broker.pdb.create (gt $replicaCount 0) }}
{{- if .Values.broker.pdb.create }}
apiVersion: {{ include "common.capabilities.policy.apiVersion" . }}
kind: PodDisruptionBudget
metadata:
@@ -20,8 +19,8 @@ spec:
{{- if .Values.broker.pdb.minAvailable }}
minAvailable: {{ .Values.broker.pdb.minAvailable }}
{{- end }}
{{- if .Values.broker.pdb.maxUnavailable }}
maxUnavailable: {{ .Values.broker.pdb.maxUnavailable }}
{{- if or .Values.broker.pdb.maxUnavailable (not .Values.broker.pdb.minAvailable) }}
maxUnavailable: {{ .Values.broker.pdb.maxUnavailable | default 1 }}
{{- end }}
{{- $podLabels := include "common.tplvalues.merge" ( dict "values" ( list .Values.broker.podLabels .Values.commonLabels ) "context" . ) }}
selector:

View File

@@ -4,7 +4,7 @@ SPDX-License-Identifier: APACHE-2.0
*/}}
{{- $replicaCount := int .Values.broker.replicaCount }}
{{- if gt $replicaCount 0 }}
{{- if or (gt $replicaCount 0) .Values.broker.autoscaling.enabled }}
apiVersion: {{ include "common.capabilities.statefulset.apiVersion" . }}
kind: StatefulSet
metadata:
@@ -18,7 +18,9 @@ metadata:
{{- end }}
spec:
podManagementPolicy: {{ .Values.broker.podManagementPolicy }}
{{- if not .Values.broker.autoscaling.enabled }}
replicas: {{ .Values.broker.replicaCount }}
{{- end }}
{{- $podLabels := include "common.tplvalues.merge" ( dict "values" ( list .Values.broker.podLabels .Values.commonLabels ) "context" . ) }}
selector:
matchLabels: {{- include "common.labels.matchLabels" ( dict "customLabels" $podLabels "context" $ ) | nindent 6 }}

View File

@@ -3,8 +3,7 @@ Copyright Broadcom, Inc. All Rights Reserved.
SPDX-License-Identifier: APACHE-2.0
*/}}
{{- $replicaCount := int .Values.controller.replicaCount }}
{{- if and .Values.controller.pdb.create .Values.kraft.enabled (gt $replicaCount 0) }}
{{- if and .Values.controller.pdb.create .Values.kraft.enabled }}
apiVersion: {{ include "common.capabilities.policy.apiVersion" . }}
kind: PodDisruptionBudget
metadata:
@@ -20,8 +19,8 @@ spec:
{{- if .Values.controller.pdb.minAvailable }}
minAvailable: {{ .Values.controller.pdb.minAvailable }}
{{- end }}
{{- if .Values.controller.pdb.maxUnavailable }}
maxUnavailable: {{ .Values.controller.pdb.maxUnavailable }}
{{- if or .Values.controller.pdb.maxUnavailable (not .Values.controller.pdb.minAvailable) }}
maxUnavailable: {{ .Values.controller.pdb.maxUnavailable | default 1 }}
{{- end }}
{{- $podLabels := include "common.tplvalues.merge" ( dict "values" ( list .Values.controller.podLabels .Values.commonLabels ) "context" . ) }}
selector:

View File

@@ -4,7 +4,7 @@ SPDX-License-Identifier: APACHE-2.0
*/}}
{{- $replicaCount := int .Values.controller.replicaCount }}
{{- if and .Values.kraft.enabled (gt $replicaCount 0) }}
{{- if and .Values.kraft.enabled (or (gt $replicaCount 0) .Values.controller.autoscaling.enabled) }}
apiVersion: {{ include "common.capabilities.statefulset.apiVersion" . }}
kind: StatefulSet
metadata:
@@ -18,7 +18,9 @@ metadata:
{{- end }}
spec:
podManagementPolicy: {{ .Values.controller.podManagementPolicy }}
{{- if not .Values.controller.autoscaling.enabled }}
replicas: {{ .Values.controller.replicaCount }}
{{- end }}
{{- $podLabels := include "common.tplvalues.merge" ( dict "values" ( list .Values.controller.podLabels .Values.commonLabels ) "context" . ) }}
selector:
matchLabels: {{- include "common.labels.matchLabels" ( dict "customLabels" $podLabels "context" $ ) | nindent 6 }}

View File

@@ -845,9 +845,9 @@ controller:
## @param controller.pdb.maxUnavailable Maximum number/percentage of unavailable Kafka replicas
##
pdb:
create: false
create: true
minAvailable: ""
maxUnavailable: 1
maxUnavailable: ""
## Enable persistence using Persistent Volume Claims
## ref: https://kubernetes.io/docs/concepts/storage/persistent-volumes/
##
@@ -1255,9 +1255,9 @@ broker:
## @param broker.pdb.maxUnavailable Maximum number/percentage of unavailable Kafka replicas
##
pdb:
create: false
create: true
minAvailable: ""
maxUnavailable: 1
maxUnavailable: ""
## @section Experimental: Kafka Broker Autoscaling configuration
## ref: https://kubernetes.io/docs/tasks/run-application/horizontal-pod-autoscale/
##
@@ -2068,6 +2068,7 @@ metrics:
## @param metrics.prometheusRule.groups Prometheus Rule Groups for Kafka
##
groups: []
## @section Kafka provisioning parameters
##