Add ReserveSpace parameter to StorageClass

[Foxite]

Summary

• Updated the helm chart and example in values.yaml for Optional space reservation (thin provisioning) #102.

Checklist

For Helm Chart changes

• 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
• PR contains the label area:chart
• PR contains the chart label, e.g. chart:kubernetes-zfs-provisioner
• Variables are documented in the values.yaml using the format required by Helm-Docs.
• Chart Version bumped if immediate release after merging is planned
• I have run make chart-docs
• Link this PR to related code release or other issues.

[Foxite]

Do you think this requires manual testing?

[ccremer]

Do you think this requires manual testing?

If you like, you can add unit test in Go to see if the parameters applied are rendered correctly, see the test folder within the chart.
https://github.com/ccremer/kubernetes-zfs-provisioner/blob/master/charts/kubernetes-zfs-provisioner/test/storageclass_test.go

[ccremer]

Thanks for the PR :)

Have you run the command to generate the Chart readme? It seems to be missing, as well as the version bump in chart.yaml

[Foxite]

I just ran make chart-docs but nothing changed.

What version should I bump to?

[ccremer]

What version should I bump to?

Since this is a new feature, version 2.2.0 would be next
https://github.com/ccremer/kubernetes-zfs-provisioner/blob/master/charts/kubernetes-zfs-provisioner/Chart.yaml

I just ran make chart-docs but nothing changed.

It should update the readme after you bump the version IIRC

[Foxite]

All done!

[ccremer]

Merged & released.
Thanks for the PRs :)