Postpone Deletion of a Persistent Volume Claim in case It Is Used by a Pod #6415

pospispa · 2017-11-22T19:32:57Z

Documentation for PRs:

Target release: 1.9

This change is

k8sio-netlify-preview-bot · 2017-11-22T19:35:05Z

Deploy preview ready!

Built with commit 99e1b08

https://deploy-preview-6415--kubernetes-io-vnext-staging.netlify.com

pospispa · 2017-11-22T19:35:09Z

/milestone 1.9

pospispa · 2017-11-24T15:20:21Z

/sig storage
/assign @msau42
/assign @jsafrane

pospispa · 2017-11-24T15:22:33Z

@msau42 @jsafrane PTAL

msau42 · 2017-11-28T01:42:59Z

Add this tag at the top of each new area:

{% assign for_k8s_version="v1.9" %}{% include feature-state-alpha.md %}

pospispa · 2017-11-28T11:16:02Z

@msau42

Add this tag at the top of each new area:

{% assign for_k8s_version="v1.9" %}{% include feature-state-alpha.md %}

Done

jsafrane · 2017-11-28T11:30:11Z

docs/tasks/administer-cluster/pvc-protection.md

+{% capture overview %}
+{% assign for_k8s_version="v1.9" %}{% include feature-state-alpha.md %}
+
+As of Kubernetes 1.9, persistent volume claims (PVCs) that are actively used by a pod can be protected from pre-mature delettion.


typo: delettion

jsafrane · 2017-11-28T11:30:30Z

docs/tasks/administer-cluster/pvc-protection.md

+## PVC Protection Configuration
+
+As PVC Protection is an alpha feature it must be turned on:
+1. Admission controller must be started with the PVC Protection plugin.


Link to admission plugin docs would be handy

jsafrane · 2017-11-28T11:50:03Z

docs/tasks/administer-cluster/pvc-protection.md

+
+## PVC Protection Configuration
+
+As PVC Protection is an alpha feature it must be turned on:


The list below is list of installation/configuration prerequisites. It can't be configured in runtime, therefore IMO belongs to capture prerequisites section above

jsafrane · 2017-11-28T11:51:58Z

docs/tasks/administer-cluster/pvc-protection.md

+
+As PVC Protection is an alpha feature it must be turned on:
+1. Admission controller must be started with the PVC Protection plugin.
+2. All kubelets must be started with the `PVCProtection` alpha features switched on.


At least API server and controller-manager must have the feature enabled too. It's better to suggest users turning it on in all Kubernetes components

jsafrane · 2017-11-28T11:54:08Z

docs/tasks/administer-cluster/pvc-protection.md

+approvers:
+- msau42
+- jsafrane
+title: Persistent Volume Claim Protection


This page is a task. Taks has steps with concrete examples. There are no steps below, just installation prerequisites. I'd expect some PVC and Pod creation, PVC deletion (+ show that PVC is not deleted) and Pod deletion (+ show that PVC and PV is deleted).

Or don't create a task page and put the info somewhere else.

@jsafrane It looks like creation (line 31) and deletion (line 81) are both covered, so a task is the most appropriate template here.

msau42 · 2017-12-01T23:30:14Z

docs/admin/admission-controllers.md

@@ -329,6 +329,11 @@ This plug-in also protects the access to `metadata.ownerReferences[x].blockOwner
 of an object, so that only users with "update" permission to the `finalizers`
 subresource of the referenced *owner* can change it.

+### Persistent Volume Claim Protection (alpha)
+{% assign for_k8s_version="v1.9" %}{% include feature-state-alpha.md %}
+The `PVCProtection` plugin protects Persistent Volume Claims (PVCs) that are being actively used by a pod from being deleted by adding `kubernetes.io/pvc-protection` finalizer to newly created PVCs.


by adding the kubernetes.io/pvc-protection finalizer?

msau42 · 2017-12-01T23:31:41Z

docs/concepts/storage/persistent-volumes.md

@@ -70,6 +70,25 @@ Pods use claims as volumes. The cluster inspects the claim to find the bound vol

 Once a user has a claim and that claim is bound, the bound PV belongs to the user for as long as they need it. Users schedule Pods and access their claimed PVs by including a persistentVolumeClaim in their Pod's volumes block. [See below for syntax details](#claims-as-volumes).

+### Persistent Volume Claim Protection
+{% assign for_k8s_version="v1.9" %}{% include feature-state-alpha.md %}
+In case the [PVC protection alpha feature](/docs/tasks/administer-cluster/pvc-protection/) is switched on, a PVC is being actively used by a pod and a user deletes the PVC the PVC is not deleted immediately, but the PVC deletion is postponed until it is not actively used by any pods.


switched on => enabled ?

msau42 · 2017-12-01T23:32:24Z

docs/admin/admission-controllers.md

@@ -329,6 +329,11 @@ This plug-in also protects the access to `metadata.ownerReferences[x].blockOwner
 of an object, so that only users with "update" permission to the `finalizers`
 subresource of the referenced *owner* can change it.

+### Persistent Volume Claim Protection (alpha)
+{% assign for_k8s_version="v1.9" %}{% include feature-state-alpha.md %}
+The `PVCProtection` plugin protects Persistent Volume Claims (PVCs) that are being actively used by a pod from being deleted by adding `kubernetes.io/pvc-protection` finalizer to newly created PVCs.


do you need to define what "actively used by a pod" means?

msau42 · 2017-12-01T23:34:40Z

docs/tasks/administer-cluster/pvc-protection.md

+
+## PVC Protection Verification
+
+The below is done in GCE using a Storage Class, however, similar steps can be performed for any type of storage.


The example below uses a GCE PD StorageClass, however, similar steps can be performed for any volume type.

msau42 · 2017-12-01T23:35:25Z

docs/tasks/administer-cluster/pvc-protection.md

+
+There are two scenarios: a PVC that is being deleted is either used or not used by a pod.
+
+Firstly, the PVC is not used by a pod:


### Scenario 1: The PVC is not used by a pod

msau42 · 2017-12-01T23:36:29Z

docs/tasks/administer-cluster/pvc-protection.md

+There are two scenarios: a PVC that is being deleted is either used or not used by a pod.
+
+Firstly, the PVC is not used by a pod:
+- Create a PVC:


Do you want to number each step?

Well, I tried it originally and I tried it once again, but the rendering doesn't work for me. Either the numbering is screwed up (1., 1., 1. instead of 1., 2., 3.) or the verbatim (```) output is screwed up (not being verbatim at all). That's why I use the bullet points instead of numbers.

You have to indent the yamls so that the numbering doesn't reset. Take a look at this for an example.

msau42 · 2017-12-01T23:36:47Z

docs/tasks/administer-cluster/pvc-protection.md

+```
+- Delete the PVC and check that the PVC (not used by a pod) was deleted successfully.
+
+Secondly, the PVC is used by a pod:


### Scenario 2: The PVC is used by a pod

msau42 · 2017-12-01T23:38:04Z

docs/tasks/administer-cluster/pvc-protection.md

+  ----    ------                 ----  ----                         -------
+  Normal  ProvisioningSucceeded  52s   persistentvolume-controller  Successfully provisioned volume pvc-803a1f4d-d6a6-11e7-9af0-42010a800002 using kubernetes.io/gce-pd
+```
+- Wait until the pod becomes `Terminated` (either delete the pod or wait until it finishes). Afterwards, check that the PVC that is no longer used by a pod is deleted.


"check that the PVC is deleted" is more concise

…ion description in PVC lifecycle.

pospispa · 2017-12-04T12:58:01Z

/retest

msau42 · 2017-12-04T17:57:20Z

docs/concepts/storage/persistent-volumes.md

+{% assign for_k8s_version="v1.9" %}{% include feature-state-alpha.md %}
+The purpose of the PVC protection is to ensure that PVCs in active use by a pod are not removed from the system as this may result in data loss.
+
+Note: PVC is in active use by a pod when the pod status is `Running`.


What about when it has been scheduled to a node, but node hasn't finished bringing it to running?

The tipping point is when a pod is assigned to a node.

I thought that the only thing a user sees are pod phases that do not have a separate phase for a pod that is not assigned to a node and for a pod that is assigned to a node. That's why I didn't include the Pending phase.

But it would be better to include the state when a pod is in Pending phase and is assigned to a node.

tengqm · 2017-12-05T01:25:56Z

docs/tasks/administer-cluster/pvc-protection.md

+- A v1.9 or higher Kubernetes must be installed.
+- As PVC Protection is a Kubernetes v1.9 alpha feature it must be enabled:
+1. [Admission controller](/docs/admin/admission-controllers/) must be started with the [PVC Protection plugin](/docs/admin/admission-controllers/#persistent-volume-claim-protection-alpha).
+2. All Kubernetes components must be started with the `PVCProtection` alpha features enabled.


IIRC, the feature is about apiserver, controller-manager and kubelet. Enabling it on scheduler or kube-proxy components are not needed.

Assuming the approved scheduler changes won't make it into K8s 1.9 it won't be necessary to enable it on scheduler and kube-proxy components.

If the scheduler changes had made it into K8s 1.9 it would be necessary to enable it also on scheduler component.

Anyway, in beta version it will be necessary to enable it also on scheduler.

However, as @jsafrane suggested in his comment it's better to suggest users to enable it on all components.

As the feature applies to majority of K8s components I also prefer to suggest users to enable it on all components.

pospispa · 2017-12-05T12:49:22Z

@msau42 @zacharysarah I hope I've addressed all your comments.

zacharysarah · 2017-12-06T21:02:59Z

Docs LGTM 👍

@msau42 @jsafrane Are these Tech LGTM?

tengqm · 2017-12-07T01:33:43Z

/lgtm

* Trivial change to open release branch * Undo trivial change * add service ipvs overview * Add instructions on how to setup kubectl * Document conntrack dependency for kube-proxy * Add an a This is kind of jarring / missing an article. I'm guessing it should either be ' to a rack of bare metal servers.' or '...to racks of bare metal servers.'. * adding example responses for common issues - support request - code bug report * Trivial change to open release branch * Undo trivial change * Signed-off-by: Ziqi Zhao <zhaoziqi@qiniu.com> (#5366) Fix the not-working test case yaml for /doc/concepts/storage/volumes.md * kubectl-overview * temp fix for broken pod and deployment links * Update Table of Solutions for Juju * Revise certificates documentation (#5965) * Update review-issues.md Some edits for clarity and condensed language. * Update init-containers.md Fix leading spaces in commands. * Update kubectl-overview.md Fix format. * Update clc.md Fix format. * Update openstack-heat.md The url no need. just highlight. * Typo I believe this should be "users" not "uses" * making explicit hostname uniq requirement * Update scheduling-hugepages.md * Update update-daemon-set.md * fix redirection of PersistentVolume * Update hpa.md * update kubectl instruction * Use the format of kubeadm init * fix spelling error guarnatees to guarantees * add matchLabels description (#6020) * search and replace for k8s.github.io to website (#6019) * fix scale command of object-management (#6011) * Update replicaset.md (#6009) * Update secret.md (#6008) * specify password for mysql image (#5990) * specify password for mysql image * specify password for mysql image * link error for run-stateless-application-deployment.md (#5985) * link error for run-stateless-application-deployment.md * link error for run-stateless-application-deployment.md * Add performance implications of inter-pod affinity/anti-affinity (#5979) * 404 monthly maintenance - October 2017 (#5977) * Updated redirects * More redirects * Add conjure-up to Turnkey Cloud Solutions list (#5973) * Add conjure-up to Turnkey Cloud Solutions list * Changed wording slightly * change the StatefulSet to ReplicaSet in reference (#5968) * Clarification of failureThreshold of probes (#5963) * Mention usage of block storage version param (#5925) Mention usage of block storage version (bs-version) parameter to workaround attachment issues using older K8S versions on an OpenStack cloud with path-based endpoints. Resolves: #5924 * Update sysctl-cluster.md (#5894) Include guide on enabling unsafe sysctls in minikube * Avoid Latin phrases & format note (#5889) * Avoid Latin phrases & format note according the Documentation Style Guide * Update scratch.md * Update scratch.md * resolves jekyll rendering error (#5976) - chinese isn't understood for keys in YAML frontmatter in jekyll, so replaced it with the english equivalent that doesn't throw the following error on rendering: Error reading file src/kubernetes.github.io/cn/docs/concepts/cluster-administration/device-plugins.md: (<unknown>): could not find expected ':' while scanning a simple key at line 4 column 1 * Change VM to pod. (#6022) * Add link to custom metrics. (#6023) * Rephrase core group. (#6024) * Added explanation on context to when joining (#6018) * Update create-cluster-kubeadm.md (#5761) Update Canal version in pod network apply commands * Fixes issue #5620 (#5869) * Fixes issue #5620 Signed-off-by: Brad Topol <btopol@us.ibm.com> * Restructured so that review process is for both current and upcoming releases. Added content describing the use of tech reviewers. * Removed incorrect Kubernetes reviewer link. * Fixed tech reviewer URL to now use website * Update pod-priority-preemption.md fix-wrong-link-to-pod-preemption * pod-security-policy.md: add links to the page about admission plugins. * Adding all files for BlaBlaCar case study (#5857) * Adding all files for BlaBlaCar case study * Update blablacar.html * Fix changed URL for google containers * Add /docs/reference/auto-generated directory * correct the downwardapi redirect * Remove links using "here" * Rename to /docs/reference/generated directory * add Concept template * Change title to just Ingress * Link mistake (#6038) * link mistake * link mistake * skip title check for skip_title_check.txt * skip title check for skip_title_check.txt * remove doesn't exist link. * Fix podpreset task (#5705) * Add a simple pod manifest to pod overview (#5986) * Split PodPreset concept out from task doc (#5984) * Add selector spec description (#5789) * Add selector spec description * Fix selector field explanation * Put orphaned topics in TOC. (#6051) * static-pod example bad format in the final page (#6050) * static-pod example bad format in the final page * static-pod example bad format in the final page * static-pod example bad format in the final page * static-pod example bad format in the final page * static-pod example bad format in the final page * Fix `backoffLimit` field misplacement (#6042) It should be placed in JobSpec according to: https://github.com/kubernetes/kubernetes/blob/master/api/swagger-spec/batch_v1.json#L1488-L1514 * Update addons.md (#6061) * add info about VMware NSX-T CNI plugin (#5987) * add info about VMware NSX-T CNI plugin Hello, I'm VMware Networking and Security Architect and would like to include short information about our CNI plugin implementation similar to what other vendors did Best regards Emil Gagala * Update networking.md * Update networking.md * Update networking.md * Update: Using universal zsh configuration (#5669) * Update install-kubectl.md Zsh is not only oh-my-zsh, so I added universal configuration for zsh that also can be used in prezto. * fix merge error after rebase * Operating etcd cluster for Kubernetes bad format in the final page (#6056) * Operating etcd cluster for Kubernetes bad format in the final page * Update configure-upgrade-etcd.md * Update configure-upgrade-etcd.md * Usage note and warning tags. (#6053) * Usage note and warning tags. * Update configure-upgrade-etcd.md * Update configure-upgrade-etcd.md * Document jekyll includes snippets * Add jekyll includes to docs home toc - Remove extra kubernetes home in toc * document docker cgroupdriver req (#5937) * Update test blacklists (#6063) * Update toc check blacklist * Update title check blacklist * wip * wip * Fix typo * Document unconfined apparmor profile * Revert "Document the unconfined profile for AppArmor" (#6268) * CRD Validation: remove alpha warning, change enable instructions to (#6066) disable * Documented service annotation for AWS ELB SSL policy * kubeadm: add a note about the new `--print-join-command` flag. This is a new flag for the `kubeadm token create` command. * Add a note to PDB page * Improve Kubeadm reference doc (#6103) * automatically-generated kubeadm reference doc * user-mantained kubeadm reference doc * Documentation for CSIPersistentVolume * change replicaset documentation to use apps/v1 APIs * Update service.md ipvs alpha version -> beta version * Updated Deployment concept docs (#6494) * Updated Deployment concept docs * Addressed comments * Documentation for volume scheduling alpha feature * Update admission control docs for webhooks * Improve DNS documentation (#6479) * update ds for 1.9 * Update service.md * Update service.md * Revert "begin updating webhook documentation" (#6575) * Update version numbers to include 1.9 (#6518) * Update site versions for 1.9 * Removed 1.4 docs * Update _config.yml * Update _config.yml * updates for raw block devices * rbac: docs for aggregated cluster roles (#6474) * Added IPv6 information for Kubelet arguments (#6498) * Added IPv6 info to kube-proxy arguments * Added IPv6 information for argument for kubelet * Update PVC resizing documentation (#6487) * Updates for Windows Server version 1709 with K8s v1.8 (#6180) * Updated for WSv1709 and K8s v1.8 * Updated picture and CNI config * Fixed formatting on CNI Config * Updated docs to reference Microsoft/SDN GitHub docs * fix typo * Workaround for Jekyllr frontmatter * Added section on features and limitations, with example yaml files. * Update index.md * Added kubeadm section, few other small fixes * Few minor grammar fixes * Update access-cluster.md with a comment that for IPv6 the user should use [::1] for the localhost * Addressed a number of issues brought up against the base PR * Fixed windows-host-setup link * Rewrite PodSecurityPolicy guide * Update index.md Signed-off-by: Alin Balutoiu <abalutoiu@cloudbasesolutions.com> Signed-off-by: Alin Gabriel Serdean <aserdean@ovn.org> * Spelling correction and sentence capitalization. - Corrected the spelling error for storing, was put in as 'stoing'. - Capitalized list items. - Added '.' at end of sentences in the list items. * Update index.md * Update index.md * Addressed comments and rebased * Fixed formatting * Fixed formatting * Updated header link * Updated hyperlinks * Updated warning * formatting * formatting * formatting * Revert "Update access-cluster.md with a comment that for IPv6" This reverts commit 31e4dbd. * Revert "fix typo" This reverts commit c056787. * Revert "Workaround for Jekyllr frontmatter" This reverts commit b84ac59. * Fixed grammatical issues and reverted non-related commits * Revert "Rewrite PodSecurityPolicy guide" This reverts commit 5d39cfe. * Revert "Spelling correction and sentence capitalization." This reverts commit 47eed43. * Fixed auto-numbering * Minor formatting updates * CoreDNS feature documentation (#6463) * Initial placeholder PR for CoreDNS feature documentation * Remove from admin, add content * Fix missing endcapture * Add to tasks.yml * Review feedback * Postpone Deletion of a Persistent Volume Claim in case It Is Used by a Pod (#6415) * Postpone Deletion of a Persistent Volume Claim in case It Is Used by a Pod A new feature PVC Protection was added into K8s 1.9 that's why this documentation change is needed. * Added tag at the top of each new area. * Fix typo * Fix: switched on in (all kubelets) -> (all K8s components). * Added link to admission controller * Moved PVC Protection configuration into Before you begin section. * Added steps how to verify PVC Protection feature. * Fixes for admission controller plugin description and for PVC Protection description in PVC lifecycle. * Testing official rendering of enumerations (1., 2., 3., etc.) * Re-write to address comments from review. * Fixed definition when a PVC is in active use by a pod. * Change auditing docs page for 1.9 release (#6427) * Change auditing docs page for 1.9 release Signed-off-by: Mik Vyatskov <vmik@google.com> * Address review comments Signed-off-by: Mik Vyatskov <vmik@google.com> * Address review comments Signed-off-by: Mik Vyatskov <vmik@google.com> * Address review comments Signed-off-by: Mik Vyatskov <vmik@google.com> * Fix broken link Signed-off-by: Mik Vyatskov <vmik@google.com> * short circuit deny docs (#6536) * line wrap * short circuit deny * address comments * Add kubeadm 1.9 upgrade docs (#6485) * kubeadm: Improve kubeadm documentation for v1.9 (#6645) * Update admission control docs for webhooks (re-send #6368) (#6650) * Update admission control docs for webhooks * update in response to comments * Revamp rkt and add CRI-O as alternative runtime (#6371) Signed-off-by: Lorenzo Fontana <lo@linux.com> * Documented NLB for Kubernetes 1.9 (#6260) * Added IPV6 information to setup cluster using kubeadm (#6465) * Added IPV6 information to setup cluster using kubeadm * Updated kubeadm.md & create-cluster-kubeadm.md with IPv6 related information * Added IPv6 options for kubeadm --init & automated address binding for kube-proxy based on version of IP configured for API server) * Changes to kubeadm.md as per comments * Modified kubeadm.md and create-cluster-kubeadm.md * Implemented changes requested by zacharysarah * Removed autogenerated kubeadm.md changes * StatefulSet 1.9 updates. (#6550) * updates sts concept and tutorials to use 1.9 apps/v1 * Update statefulset.md * clarify pod name label * Garbage collection updates for 1.9 (#6555) * 1.9 gc policy update * carify deletion * Couple nits for dnsConfig doc (#6652) * Add doc for AllowedFlexVolume (#6563) * Update OpenStack Cloud Provider API support for v1.9 (#6638) * Flex volume is GA. Remove alpha notation. (#6666) * Update generated ref docs for Kubernetes and Federation components. (#6658) * Update generated ref docs for Kubernetes and Federation components. * Rename kubectl-options to kubectl. * Add title to kubectl. * Fix double synopsis. * Update Federation API ref docs for 1.9. (#6636) * Update federation API ref docs. * Move and redirect. * Move generated Federation docs to the generated directory. * Fix titles. * Type * Fix titles * Update auto-generated Kubernetes APi ref docs. (#6646) * Update kubectl commands for 1.9 (#6635) * add ExtendedResourceToleration admission controller (#6618) * Update API reference paths for v1.9 (#6681)

PVC Protection alpha feature was added to K8s 1.9 in PRs: - kubernetes/kubernetes#55824 - kubernetes/kubernetes#55873 and documented in K8s 1.9 in PR: - kubernetes/website#6415 That's why the PVC Protection is a new alpha feature in OpenShift 3.9 and that's why PVC Protection documentation is added.

k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 22, 2017

k8s-github-robot assigned Bradamant3 Nov 22, 2017

pospispa mentioned this pull request Nov 23, 2017

StorageObjectInUseProtection (was Persistent Volume Claim Protection) kubernetes/enhancements#498

Closed

pospispa force-pushed the 566-postpone-pvc-deletion-if-used-in-a-pod branch from 61abef6 to a54be1f Compare November 24, 2017 13:53

k8s-ci-robot added size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Nov 24, 2017

pospispa force-pushed the 566-postpone-pvc-deletion-if-used-in-a-pod branch from a54be1f to 15caa0a Compare November 24, 2017 15:01

pospispa changed the title ~~WIP: Postpone Deletion of a Persistent Volume Claim in case It Is Used by a Pod~~ Postpone Deletion of a Persistent Volume Claim in case It Is Used by a Pod Nov 24, 2017

k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Nov 24, 2017

pospispa force-pushed the 566-postpone-pvc-deletion-if-used-in-a-pod branch from 15caa0a to 9e55e18 Compare November 24, 2017 15:13

k8s-ci-robot assigned jsafrane and msau42 Nov 24, 2017

zacharysarah added this to In Progress in Release 1.9 Nov 27, 2017

steveperry-53 added this to the 1.9 milestone Nov 28, 2017

steveperry-53 added Needs Docs Review labels Nov 28, 2017

pospispa force-pushed the 566-postpone-pvc-deletion-if-used-in-a-pod branch from 4e747b6 to b82ff4c Compare November 28, 2017 11:07

jsafrane reviewed Nov 28, 2017

View reviewed changes

zacharysarah moved this from In Progress to To Do in Release 1.9 Nov 29, 2017

steveperry-53 added Tech Review: Open Issues and removed Needs Tech Review labels Nov 29, 2017

zacharysarah mentioned this pull request Nov 30, 2017

Release 1.9 #5978

Merged

zacharysarah moved this from To Do to In Progress in Release 1.9 Dec 1, 2017

msau42 reviewed Dec 1, 2017

View reviewed changes

pospispa added 2 commits December 4, 2017 11:49

Fixes for admission controller plugin description and for PVC Protect…

914c4ed

…ion description in PVC lifecycle.

Testing official rendering of enumerations (1., 2., 3., etc.)

5515c0f

Re-write to address comments from review.

6e66355

pospispa force-pushed the 566-postpone-pvc-deletion-if-used-in-a-pod branch from df0a2f6 to 6e66355 Compare December 4, 2017 13:00

msau42 reviewed Dec 4, 2017

View reviewed changes

tengqm reviewed Dec 5, 2017

View reviewed changes

Fixed definition when a PVC is in active use by a pod.

99e1b08

zacharysarah approved these changes Dec 6, 2017

View reviewed changes

zacharysarah added Docs LGTM and removed Docs Review: Open Issues labels Dec 6, 2017

k8s-ci-robot assigned tengqm Dec 7, 2017

k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Dec 7, 2017

zacharysarah mentioned this pull request Dec 7, 2017

[1.9] Issue Burndown kubernetes/sig-release#38

Closed

zacharysarah added Tech Review LGTM and removed Tech Review: Open Issues labels Dec 7, 2017

zacharysarah merged commit a6eb858 into kubernetes:release-1.9 Dec 7, 2017

zacharysarah moved this from In Progress to Done in Release 1.9 Dec 11, 2017

aburdenthehand mentioned this pull request Dec 13, 2017

PVCProtection feature not documented #6506

Closed

pospispa mentioned this pull request Jan 15, 2018

Backport PVC Protection Feature Scheduler Changes openshift/origin#18110

Closed

pospispa mentioned this pull request Feb 2, 2018

Persistent Volume Claim Protection openshift/openshift-docs#7497

Closed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Postpone Deletion of a Persistent Volume Claim in case It Is Used by a Pod #6415

Postpone Deletion of a Persistent Volume Claim in case It Is Used by a Pod #6415

pospispa commented Nov 22, 2017 •

edited

Loading

k8sio-netlify-preview-bot commented Nov 22, 2017 •

edited

Loading

pospispa commented Nov 22, 2017

pospispa commented Nov 24, 2017

pospispa commented Nov 24, 2017

msau42 commented Nov 28, 2017

pospispa commented Nov 28, 2017

jsafrane Nov 28, 2017

jsafrane Nov 28, 2017

jsafrane Nov 28, 2017

jsafrane Nov 28, 2017

jsafrane Nov 28, 2017 •

edited

Loading

zacharysarah Dec 1, 2017

msau42 Dec 1, 2017

msau42 Dec 1, 2017

msau42 Dec 1, 2017

msau42 Dec 1, 2017

msau42 Dec 1, 2017

msau42 Dec 1, 2017

pospispa Dec 4, 2017

msau42 Dec 4, 2017

msau42 Dec 1, 2017

msau42 Dec 1, 2017

pospispa commented Dec 4, 2017

msau42 Dec 4, 2017

pospispa Dec 5, 2017

tengqm Dec 5, 2017

pospispa Dec 5, 2017

pospispa commented Dec 5, 2017

zacharysarah commented Dec 6, 2017

tengqm commented Dec 7, 2017


		## PVC Protection Configuration

		As PVC Protection is an alpha feature it must be turned on:


		## PVC Protection Verification

		The below is done in GCE using a Storage Class, however, similar steps can be performed for any type of storage.


		There are two scenarios: a PVC that is being deleted is either used or not used by a pod.

		Firstly, the PVC is not used by a pod:

Postpone Deletion of a Persistent Volume Claim in case It Is Used by a Pod #6415

Postpone Deletion of a Persistent Volume Claim in case It Is Used by a Pod #6415

Conversation

pospispa commented Nov 22, 2017 • edited Loading

k8sio-netlify-preview-bot commented Nov 22, 2017 • edited Loading

pospispa commented Nov 22, 2017

pospispa commented Nov 24, 2017

pospispa commented Nov 24, 2017

msau42 commented Nov 28, 2017

pospispa commented Nov 28, 2017

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jsafrane Nov 28, 2017 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

pospispa commented Dec 4, 2017

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

pospispa commented Dec 5, 2017

zacharysarah commented Dec 6, 2017

tengqm commented Dec 7, 2017

pospispa commented Nov 22, 2017 •

edited

Loading

k8sio-netlify-preview-bot commented Nov 22, 2017 •

edited

Loading

jsafrane Nov 28, 2017 •

edited

Loading