This is a change for backwards compatibility reasons. Previously existing storage classes can't be modified with new parameters. Thus, we only render it if it has been explicitly set (e.g. when rendering a new storage class). The controller defaults the value to 'true' internally, so omitting the parameter is fine.
Follow-up of #103
Checklist
[x] Categorize the PR by setting a good title and adding one of the labels:
kind:bug, kind:enhancement, kind:documentation, kind:change, kind:breaking, kind:dependency
as they show up in the changelog
[x] PR contains the label area:chart
[x] PR contains the chart label, e.g. chart:kubernetes-zfs-provisioner
[x] Variables are documented in the values.yaml using the format required by Helm-Docs.
[x] Chart Version bumped if immediate release after merging is planned
[x] I have run make chart-docs
[x] Link this PR to related code release or other issues.
Summary
This is a change for backwards compatibility reasons. Previously existing storage classes can't be modified with new parameters. Thus, we only render it if it has been explicitly set (e.g. when rendering a new storage class). The controller defaults the value to 'true' internally, so omitting the parameter is fine.
Follow-up of #103
Checklist
kind:bug
,kind:enhancement
,kind:documentation
,kind:change
,kind:breaking
,kind:dependency
as they show up in the changelogarea:chart
chart:kubernetes-zfs-provisioner
make chart-docs