IBM / core-dump-handler

Save core dumps from a Kubernetes Service or RedHat OpenShift to an S3 protocol compatible object store
https://ibm.github.io/core-dump-handler/
MIT License
131 stars 40 forks source link

Chart value `bindPVThroughClaimRef` to control how PV / PVC pairs are bound #149

Open unlightable opened 11 months ago

unlightable commented 11 months ago

Pre-binding PVC to PV through claimRef works fine on creation. But it seems like editing PV manifest afterwards will break things (especially in case of removing spec.claimRef.uid). E. g. on my kubernetes provider it unbinds PVC and errors it out due to multiple bindings to same PV.

This becomes an issue if the chart is under some gitops solution (like Argo CD) that will continuously apply manifests from chart re-renders.

Fortunately, everything works fine when using volumeName reference in PVC. Although this way PV could be bound to something else in case of storageClass collision, it doesn't seem like a big issue to me.