Add option to delete PVCs on chart uninstall

[andrzej-stencel]

Currently - in chart v2.2.0 - when chart is uninstalled, the Fluentd pods' PersistentVolumeClaim resources are left behind in the cluster.

I would expect to either always have them deleted, or - if that's undesirable - have an option to delete them on chart uninstallation. It could be the sumologic.cleanupEnabled option or a different one if required.

[andrzej-stencel]

The plan is to change the behavior of the chart to delete the PVCs on chart uninstallation. This is a breaking change that will be made for v3.0.

Note that Kubernetes has added a feature around this as alpha in v1.23 - https://kubernetes.io/blog/2021/12/16/kubernetes-1-23-statefulset-pvc-auto-deletion.

[sumo-drosiek]

Feature is not globally available yet and require feature flag to be enabled: https://kubernetes.io/docs/concepts/workloads/controllers/statefulset/#persistentvolumeclaim-retention

[sumo-drosiek]

Related helm issue: helm/helm#5156

[github-actions]

This issue is stale because it has been open for 90 days with no activity. Remove stale label or comment or this will be closed in 14 days.

[github-actions]

This issue is being closed because it has been stale for 14 days with no activity.