Support for definining storageClassNames about charts HOT 6 CLOSED

First thing I'm noticing is how the config of these storageclass has been spread throughout the different components and everytime it has been tackled in a different way.

In prometheus its handled inside the pvc-template
In zookeeper and bookkeeper its handled in the helpers but each in its own way (print functions vs full include defines.

I'm working on just adding the storageClassName now but maybe we should move towards a unified and more concise way of handling this? Define these classes in the same way across all components.

I have always enjoyed bitnami charts and how they do the defining of storage classes
Reference:
https://github.com/bitnami/charts/blob/007da4ff5a6b80968525a324219645c668cd6fed/bitnami/influxdb/templates/_helpers.tpl#L270
https://github.com/bitnami/charts/blob/007da4ff5a6b80968525a324219645c668cd6fed/bitnami/influxdb/values.yaml#L529

If we would adapt such a style this would mean quite a big overhaul.

from charts.

Ive made a PR with just the addition of the storageClassName values for now, don't have any spare time at the moment to overhaul/streamline volume config throughout the chart.

from charts.

@MisterTimn Yes! That sounds like a good idea. Would you like to make a contribution?

from charts.

@MisterTimn Looks good to me! Let's move towards that direction.

from charts.

@MisterTimn Thank you!

from charts.

This is fixed by #127

from charts.