Bug 1806563

Summary:	Validate usage of wipe_lv over multiple users of this function
Product:	[Community] LVM and device-mapper	Reporter:	Zdenek Kabelac <zkabelac>
Component:	lvm2	Assignee:	Zdenek Kabelac <zkabelac>
lvm2 sub component:	liblvm2cmd	QA Contact:	cluster-qe <cluster-qe>
Status:	NEW ---	Docs Contact:
Severity:	unspecified
Priority:	unspecified	CC:	agk, heinzm, jbrassow, prajnoha, tasleson, zkabelac
Version:	unspecified	Flags:	pm-rhel: lvm-technical-solution? pm-rhel: lvm-test-coverage?
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:		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:
Bug Depends On:
Bug Blocks:	1805892

Description Zdenek Kabelac 2020-02-24 14:29:42 UTC

Description of problem:

Usage of wipe_lv() has couple issues that should be validate across several callers of this function:

1. Error handling of failed wiping.

2. Unified reaction on 'prompt'  [n] answer.
   (Should this abort() whole command, or just skip wiping - this can confuse users.)

3. Possibly a single function for wiping a list of LVs.

(TODO in future: when wiping multiple new LVs - single prompt should be there)

Version-Release: 2.03