Skip to content

Commit b931bb1

Browse files
fix(helm): fix invalid YAML syntax and nil pointer in values.yaml
Updated domainLabels to be an empty array. Issue: The ceph-csi-rbd helm chart fails to install with the default values.yaml due to two issues: YAML Syntax Error: The domainLabels key was defined with both an empty list [] and nested list items, which is invalid YAML and causes a parsing error: error converting YAML to JSON: yaml: line 373: did not find expected key. Template Nil Pointer: The readAffinity block was commented out by default. However, the nodeplugin-daemonset.yaml template attempts to access .Values.readAffinity.enabled without a conditional check, leading to a nil pointer evaluation error. Changes: Fixed domainLabels syntax by removing the [] and providing the labels as valid, commented-out examples. Uncommented the readAffinity block and set enabled: false by default to ensure the Helm templates can render successfully. Signed-off-by: Keshav Soni <102344018+KeshavSoni2511@users.noreply.github.com>
1 parent 8b9bc61 commit b931bb1

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

charts/ceph-csi-rbd/values.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -339,7 +339,7 @@ topology:
339339
# for CSI nodeplugins to advertise their domains
340340
# NOTE: the value here serves as an example and needs to be
341341
# updated with node labels that define domains of interest
342-
domainLabels: []
342+
domainLabels:
343343
# - topology.kubernetes.io/region
344344
# - topology.kubernetes.io/zone
345345

0 commit comments

Comments
 (0)