Skip to content

🐛 Fix duplicate volumeMounts in hcloud-csi controller#118

Open
guettli wants to merge 1 commit intomainfrom
fix-hcloud-csi-duplicate-volumemounts
Open

🐛 Fix duplicate volumeMounts in hcloud-csi controller#118
guettli wants to merge 1 commit intomainfrom
fix-hcloud-csi-duplicate-volumemounts

Conversation

@guettli
Copy link
Copy Markdown
Contributor

@guettli guettli commented Mar 25, 2026

Summary

  • remove the duplicated volumeMounts key from the hcloud-csi-driver container in charts/hcloud-csi/templates/controller/deployment.yaml
  • bump the chart version to 1.3.1

Why

The template currently renders volumeMounts twice in the same YAML mapping for the hcloud-csi-driver container. Helm will still render the manifest, but strict YAML consumers reject it as invalid due to the duplicate key.

This showed up downstream when rendering the chart and passing the result to a YAML parser:

mapping key "volumeMounts" already defined

The later volumeMounts block is already the complete one because it also appends controller.extraVolumeMounts, so removing the earlier block is the minimal fix.

Validation

  • helm template test charts/hcloud-csi > /tmp/hcloud-csi-template.yaml
  • parsed /tmp/hcloud-csi-template.yaml successfully with PyYAML

@guettli guettli requested a review from batistein March 25, 2026 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant