Skip to content

Commit

Permalink
Make Kafka DefaultMode YAML 1.2 Compliant
Browse files Browse the repository at this point in the history
With the YAML 1.2 spec octals must be prefixed with `0o` and not `0`.
Parsers implementing the latest spec run into issues. See discussion
here: dtolnay/serde-yaml#225

This commit makes the Kafka script mounts compliant with YAML 1.2.

Signed-off-by: Marc Schreiber <[email protected]>
  • Loading branch information
schrieveslaach committed Dec 8, 2023
1 parent 94b660b commit c22eadf
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion bitnami/kafka/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -42,4 +42,4 @@ maintainers:
name: kafka
sources:
- https://github.com/bitnami/charts/tree/main/bitnami/kafka
version: 26.4.3
version: 26.4.4
2 changes: 1 addition & 1 deletion bitnami/kafka/templates/broker/statefulset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -374,7 +374,7 @@ spec:
- name: scripts
configMap:
name: {{ include "common.names.fullname" . }}-scripts
defaultMode: 0755
defaultMode: 493
{{- if and .Values.externalAccess.enabled .Values.externalAccess.autoDiscovery.enabled }}
- name: kafka-autodiscovery-shared
emptyDir: {}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -367,7 +367,7 @@ spec:
- name: scripts
configMap:
name: {{ include "common.names.fullname" . }}-scripts
defaultMode: 0755
defaultMode: 493
{{- if and .Values.externalAccess.enabled .Values.externalAccess.autoDiscovery.enabled }}
- name: kafka-autodiscovery-shared
emptyDir: {}
Expand Down

0 comments on commit c22eadf

Please sign in to comment.