Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Support automatic PV deletion and storage cleanup with reclaimPolicy: Delete #111

Closed
pfruh opened this issue Dec 6, 2020 · 2 comments
Closed

Support automatic PV deletion and storage cleanup with reclaimPolicy: Delete #111

pfruh opened this issue Dec 6, 2020 · 2 comments

Comments

@pfruh
Copy link

pfruh commented Dec 6, 2020

Is your feature request related to a problem?/Why is this needed
According to the example files this CSI Driver (which seems to be the only one available for NFS) only supports reclaimPolicy: Retain, if I understand correctly this means after deleting a PVC it's associated PV has to be removed manually and even after removing the PV the created files and directories are not deleted from the NFS target server.

Describe the solution you'd like in detail
I'd love if this driver could implement reclaimPolicy: Delete and proper cleanup of PVs, so when a PVC is deleted, the associated PV and files/folders on the NFS server would be deleted as well.

Describe alternatives you've considered
I've thought about somehow automating the process of regularly cleaning unbound PVs and deleting the associated directories on the NFS server, but this doesn't seem like a great solution
@pfruh pfruh changed the title Support automatic PV deletion with reclaimPolicy: Delete Support automatic PV deletion and cleanup with reclaimPolicy: Delete Dec 6, 2020
@pfruh pfruh changed the title Support automatic PV deletion and cleanup with reclaimPolicy: Delete Support automatic PV deletion and storage cleanup with reclaimPolicy: Delete Dec 6, 2020
robertpeteuil pushed a commit to robpco/csi-driver-nfs that referenced this issue Dec 7, 2020
@k8s-ci-robot
Merge pull request kubernetes-csi#111 from xing-yang/update_snapshot_…
d437673 
…v1_rc

Update snapshot CRD version
robertpeteuil pushed a commit to robpco/csi-driver-nfs that referenced this issue Dec 7, 2020
@xing-yang
Squashed 'release-tools/' changes from 4aff857..c6a88c6
db4ee38 
c6a88c6 Merge pull request kubernetes-csi#113 from xing-yang/install_snapshot_controller
45ec4c6 Fix the install of snapshot CRDs and controller
5d874cc Merge pull request kubernetes-csi#112 from xing-yang/cleanup
79bbca7 Cleanup
d437673 Merge pull request kubernetes-csi#111 from xing-yang/update_snapshot_v1_rc
57718f8 Update snapshot CRD version

git-subtree-dir: release-tools
git-subtree-split: c6a88c6
@msau42
Copy link
Collaborator

msau42 commented Dec 7, 2020

This will be supported with #30

We have not released a driver yet with this change, but you can try it out on canary.

@pfruh
Copy link
Author

pfruh commented Dec 8, 2020

Ah, missed that, thanks!

@pfruh pfruh closed this as completed Dec 8, 2020
dobsonj pushed a commit to dobsonj/csi-driver-nfs that referenced this issue Oct 16, 2023
@openshift-merge-robot
Merge pull request kubernetes-csi#111 from shiftstack/merge-bot-master
984dbc9 
Merge https://github.com/kubernetes-csi/csi-driver-nfs:master into master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@pfruh @msau42