Leptopoda commented 2 months ago

Pull Request

Description of the change

Keep nextcloud PVCs when uninstalling the helm chart

Benefits

Most helm charts keep PVCs on uninstall. The current nextcloud approach is unexpected and thus may lead to data loss. Learned that the hard way :sweat_smile:

Possible drawbacks

None that I'm aware of.

Applicable issues

fixes #53

Additional information

Checklist

[x] DCO has been signed off on the commit.
[x] Chart version bumped in Chart.yaml according to semver.
[x] (optional) Variables are documented in the README.md

provokateurin commented 2 months ago

Duplicate of https://github.com/nextcloud/helm/pull/473