sig-storage-local-static-provisioner icon indicating copy to clipboard operation
sig-storage-local-static-provisioner copied to clipboard

Does it make sense to have an option to clean PV before discovering it as a new PV?

Open cofyc opened this issue 5 years ago • 22 comments

Current lifecycle:

- raw disks -> mount points -> PV (available) -> PV (bound) -> PV (released)
                               ^                                     |   
                               | <---------- clean <-----------------|                                         

New lifecycle (optional):

- raw disks -> mount points -> [clean] -> PV (available) -> PV (bound) -> PV (released)
                                          ^                                     |   
                                          | <---------- clean <-----------------|

cofyc avatar Sep 01 '19 05:09 cofyc

use cases:

  • if we clean the volume before provisioning it as a new PV into Kubernetes, it will be safe to delete the PV objects and don't worry about the data will be leaked. (https://github.com/kubernetes-sigs/sig-storage-local-static-provisioner/pull/126#issuecomment-511651492)

cofyc avatar Sep 17 '19 06:09 cofyc

another case:

  • I use cloud storage as local pv, after pvc&pod deleted, and cleanup pv successfully. But actually, cloud storage not works as expected. Then provisioner creates the pv with same name and path. Later another pod uses the pv, it finds some data not belong to itself.

so I think it really makes sense, at least, check or log if there are some data before pv available.

yuyulei avatar Nov 25 '19 08:11 yuyulei

Yes I think this is a good idea, although this will cause the recycling flow to double for very large disks. We can add it as a new option

/help /kind feature

msau42 avatar Dec 24 '19 00:12 msau42

@msau42: This request has been marked as needing help from a contributor.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed by commenting with the /remove-help command.

In response to this:

Yes I think this is a good idea, although this will cause the recycling flow to double for very large disks. We can add it as a new option

/help /kind feature

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

k8s-ci-robot avatar Dec 24 '19 00:12 k8s-ci-robot

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta. /lifecycle stale

fejta-bot avatar Mar 23 '20 01:03 fejta-bot

/remove-lifecycle stale

cofyc avatar Mar 23 '20 01:03 cofyc

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta. /lifecycle stale

fejta-bot avatar Jun 21 '20 02:06 fejta-bot

/remove-lifecycle stale

cofyc avatar Jun 22 '20 02:06 cofyc

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta. /lifecycle stale

fejta-bot avatar Sep 20 '20 02:09 fejta-bot

/remove-lifecycle stale

Bessonov avatar Sep 28 '20 09:09 Bessonov

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-testing, kubernetes/test-infra and/or fejta. /lifecycle stale

fejta-bot avatar Dec 27 '20 10:12 fejta-bot

/remove-lifecycle stale

Bessonov avatar Dec 27 '20 10:12 Bessonov

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale

fejta-bot avatar Mar 27 '21 10:03 fejta-bot

/remove-lifecycle stale

Bessonov avatar Mar 27 '21 15:03 Bessonov

Issues go stale after 90d of inactivity. Mark the issue as fresh with /remove-lifecycle stale. Stale issues rot after an additional 30d of inactivity and eventually close.

If this issue is safe to close now please do so with /close.

Send feedback to sig-contributor-experience at kubernetes/community. /lifecycle stale

fejta-bot avatar Jun 25 '21 15:06 fejta-bot

/remove-lifecycle stale

Bessonov avatar Jun 25 '21 20:06 Bessonov

The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs.

This bot triages issues and PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue or PR as fresh with /remove-lifecycle stale
  • Mark this issue or PR as rotten with /lifecycle rotten
  • Close this issue or PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

k8s-triage-robot avatar Sep 23 '21 21:09 k8s-triage-robot

The Kubernetes project currently lacks enough active contributors to adequately respond to all issues and PRs.

This bot triages issues and PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue or PR as fresh with /remove-lifecycle rotten
  • Close this issue or PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle rotten

k8s-triage-robot avatar Oct 23 '21 21:10 k8s-triage-robot

/remove-lifecycle rotten

Bessonov avatar Oct 23 '21 21:10 Bessonov

The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs.

This bot triages issues and PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue or PR as fresh with /remove-lifecycle stale
  • Mark this issue or PR as rotten with /lifecycle rotten
  • Close this issue or PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

k8s-triage-robot avatar Jan 21 '22 22:01 k8s-triage-robot

/remove-lifecycle stale

Bessonov avatar Jan 21 '22 22:01 Bessonov

/lifecycle frozen

Bessonov avatar Jan 21 '22 22:01 Bessonov