Skip to content

Commit

Permalink
[bitnami/zookeeper] Make Zookeeper DefaultMode YAML 1.2 Compliant (bi…
Browse files Browse the repository at this point in the history
…tnami#21081)

* Make Zookeeper DefaultMode YAML 1.2 Compliant

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 Zookeeper script mount compliant with YAML 1.2.

Signed-off-by: Marc Schreiber <info@schrieveslaach.de>

* Use Decimal Notation for defaultMode

In order to be still YAML 1.1 compliant and to be YAML 1.2 compliant,
this commit changes the defaultMode to the decimal value 493. This seems
to be done in other places as well (see modes of TLS secrets).

Signed-off-by: Marc Schreiber <info@schrieveslaach.de>

---------

Signed-off-by: Marc Schreiber <info@schrieveslaach.de>
  • Loading branch information
schrieveslaach authored and cedo-l committed Jan 25, 2024
1 parent a571fbf commit a6cae69
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion bitnami/zookeeper/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -28,4 +28,4 @@ maintainers:
name: zookeeper
sources:
- https://github.com/bitnami/charts/tree/main/bitnami/zookeeper
version: 12.3.3
version: 12.3.4
2 changes: 1 addition & 1 deletion bitnami/zookeeper/templates/statefulset.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -443,7 +443,7 @@ spec:
- name: scripts
configMap:
name: {{ printf "%s-scripts" (include "common.names.fullname" .) }}
defaultMode: 0755
defaultMode: 493
{{- if or .Values.configuration .Values.existingConfigmap }}
- name: config
configMap:
Expand Down

0 comments on commit a6cae69

Please sign in to comment.