Skip to content

Helm: Wrong indentation for nodePlugin resources in driver.yaml template #435

@mohazza00

Description

@mohazza00

Describe the bug

When defining driver nodePlugin resources ($driver.nodePlugin.resources) in the values for the ceph-csi-drivers chart, we get a YAML parse error.

The reason is that we have the wrong indentation (4 instead of 8). See driver.yaml. For all these lines, we should have instead:

resources:
  registrar: {{ $driver.nodePlugin.resources.registrar | toYaml | nindent 8 }}
  liveness: {{ $driver.nodePlugin.resources.liveness | toYaml | nindent 8 }}
  addons: {{ $driver.nodePlugin.resources.addons | toYaml | nindent 8 }}
  logRotator: {{ $driver.nodePlugin.resources.logRotator | toYaml | nindent 8 }}
  plugin: {{ $driver.nodePlugin.resources.plugin | toYaml | nindent 8 }}

Environment details

  • Image/version of ceph-csi-operator: chart 0.5.0
  • Image/version of Ceph-CSI driver: chart 0.5.0
  • Kubernetes cluster version : 1.32.6
  • Ceph cluster version :

Steps to reproduce

Run helm template (helmfile template) or helm upgrade --install (helmfile apply).

Actual results

Error: YAML parse error on ceph-csi-drivers/templates/driver.yaml: error converting YAML to JSON: yaml: line 39: could not find expected ':'

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions