[stable/prometheus] Remove unnecessary reclaim policy #9471
[stable/prometheus] Remove unnecessary reclaim policy #9471
Conversation
persistentVolumeReclaimPolicy is unnecessary for PVC, so let's remove it. Signed-off-by: OTSUKA, Yuanying <mototsuk@zlab.co.jp>
cd0ea72
to
f954608
Compare
@yuanying is right. Reclaim policy was just added three days ago in PR #9284. Not sure why that was approved, as there is no field "persistentVolumeReclaimPolicy" in a PersistentVolumeClaim. |
/ok-to-test |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: gianrubio, yuanying The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
I issued #9284, and clearly I misunderstood what I was doing 😉 - I had confused PersistentVolumes and PersistentVolumeClaims. We ran into an issue where the PV was deleted after the PVC was accidentally deleted, losing us all of our Prometheus data. I thought that adding a |
persistentVolumeReclaimPolicy is unnecessary for PVC, so let's remove it. Signed-off-by: OTSUKA, Yuanying <mototsuk@zlab.co.jp>
persistentVolumeReclaimPolicy is unnecessary for PVC, so let's remove it. Signed-off-by: OTSUKA, Yuanying <mototsuk@zlab.co.jp> Signed-off-by: Chaoran Yu <yuchaoran2011@gmail.com>
persistentVolumeReclaimPolicy is unnecessary for PVC, so let's remove it. Signed-off-by: OTSUKA, Yuanying <mototsuk@zlab.co.jp>
persistentVolumeReclaimPolicy is unnecessary for PVC, so let's
remove it.
Signed-off-by: OTSUKA, Yuanying yuanying@fraction.jp
What this PR does / why we need it:
persistentVolumeReclaimPolicy is unnecessary for PVC, so let's
remove it.
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]