Ability to choose "" storageClassName

[fopina]

traefik-helm-chart/traefik/templates/pvc.yaml

Line 20 in 956a98c

{{- if .Values.persistence.storageClass }}

This check prevents me to explicitly set storageClassName: "". Ommiting class name will use default (local-path) while explicitly setting to empty will allow using nfs for instance.

I think the check should be done on whether the key is defined rather than "truthiness".

Currently working around it by using existingClaim and then extraObjects to define the whole claim from scratch.

[mloiseleur]

Thanks for your suggestion, we think it makes sense. Unfortunately, this would not make it to our roadmap for a while as we are focused elsewhere. If you or another community member would like to build it, let us know, and we will work with you to make sure you have all the information needed so that it can be merged.

Don’t forget to check out the contributor docs and to link the PR to this issue.