Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1896084

Summary:	Document removal of Finalizers on VolumeSnapshot resources if necessary
Product:	OpenShift Container Platform	Reporter:	Christian Huffman <chuffman>
Component:	Documentation	Assignee:	Bob Furu <bfuru>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Xiaoli Tian <xtian>
Severity:	unspecified	Docs Contact:	Vikram Goyal <vigoyal>
Priority:	unspecified
Version:	4.7	CC:	aos-bugs, bfuru, jokerman
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2021-03-02 23:29:08 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Christian Huffman 2020-11-09 17:21:16 UTC

This is an extension of https://bugzilla.redhat.com/show_bug.cgi?id=1893739 .

In that bug it was mentioned that `oc delete --force --grace-period=0` does not delete VolumeSnapshot or VolumeSnapshotContent resources. This is functioning as intended, as finalizers exist on these resources to prevent deletion.

We need to document removal of these finalizers, along with ensuring the note from [1] is included. Specifically:

  Note, however, that the delete operation will continue even if the secret is not 
  found (because, for example, the entire namespace containing the secret was 
  deleted). In this case, if the driver requires a secret for deletion, then the 
  volume and PV may need to be manually cleaned up.

The patch command to remove these is below:

  $ oc patch -n $PROJECT volumesnapshot/$NAME --type=merge -p '{"metadata": {"finalizers":null}}'

Please let me know if there are any questions.

[1] https://kubernetes-csi.github.io/docs/secrets-and-credentials-volume-snapshot-class.html

Comment 2 Bob Furu 2021-02-12 18:24:33 UTC

Created https://github.com/openshift/openshift-docs/pull/29475 - awaiting SME and QE review. Moving to ON_QA.

Comment 3 Bob Furu 2021-03-02 20:48:02 UTC

Approved by Eng and QE, awaiting peer review.

Comment 4 Bob Furu 2021-03-02 23:29:08 UTC

Thanks for your help, Christian.

PR has been merged and CP'ed to 4.5-4.8. Verified live on docs.openshift.com, for example: https://docs.openshift.com/container-platform/4.7/storage/container_storage_interface/persistent-storage-csi-snapshots.html#persistent-storage-csi-snapshots-delete_persistent-storage-csi-snapshots

Closing BZ.