utkuozdemir / helm-charts

Helm Chart Repository
MIT License
42 stars 14 forks source link

breaking changes from 1.x -> 2.x regarding PVC use not well documented (how to make it work on 2.x?) #11

Closed ppenguin closed 2 years ago

ppenguin commented 2 years ago

After upgrading my standard PVC defined like this (in values.yaml):

persistence:
  data:
    enabled: true
    existingClaim: pvc-nfs-microserv-bt

stopped working (now simply /data is not custom-mounted), and I couldn't find out how to make it work under 2.x.

Any ideas?

utkuozdemir commented 2 years ago

Hi,

Is this about the chart transmission-openvpn? If it is, yes, I introduced a set of breaking changes on 2.x, that's why it is a major version bump.

persistence.data moved under persistence, so persistence.data.existingClaim should re reconfigured using persistence.existingClaim. See the changes and the relevant line here: https://github.com/utkuozdemir/helm-charts/commit/c000b4151274b907f7b3843bbbab11abbc8049c6#diff-7260c926a2afd6587c0d110bc1a787469d5277341d465af77c3579b8631ba80cR84

For the record, I recommend doing a fresh install when upgrading to 2.x, see: https://github.com/utkuozdemir/helm-charts/tree/master/transmission-openvpn#from-1x-to-2x

The values is largely refactored - pay attention to the changes on the persistence section.

It is recommended to start from a fresh install.