Bug 1403839 – [RFE] Add ability to remove a single LUN from a data domain

Bugzilla will be upgraded to version 5.0. The upgrade date is tentatively scheduled for 2 December 2018, pending final testing and feedback.

Bug 1403839 - [RFE] Add ability to remove a single LUN from a data domain

Summary:	[RFE] Add ability to remove a single LUN from a data domain

Status:	CLOSED CURRENTRELEASE

Aliases:	None

Product:	ovirt-engine
Classification:	oVirt
Component:	BLL.Storage (Show other bugs)
Sub Component:
Version:	4.1.0
Hardware:	Unspecified Unspecified

Priority	unspecified Severity unspecified (vote)
Target Milestone:	ovirt-4.1.1
Target Release:	4.1.1.3
Assigned To:	Liron Aravot
QA Contact:	Lilach Zitnitski
Docs Contact:

URL:
Whiteboard:
Keywords:	FutureFeature

Depends On:	1374545 1403836
Blocks:	633978 1429746 1454287
	Show dependency tree / graph

Reported:	2016-12-12 08:09 EST by Tal Nisan
Modified:	2018-02-15 10:05 EST (History)
CC List:	7 users (show)

See Also:
Fixed In Version:
Doc Type:	Enhancement
Doc Text:	With this update, the ability to remove LUNs from a block data domain has been added. This means that LUNs can be removed from a block data domain provided that there is enough free space on the other domain devices to contain the data stored on the LUNs being removed.
Story Points:	---
Clone Of:
Clones:	1454287 (view as bug list)
Environment:
Last Closed:	2017-04-21 05:53:47 EDT
Type:	Bug
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	Storage
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Flags:	rule-engine: ovirt‑4.1+ ratamir: testing_plan_complete+ ylavi: planning_ack+ tnisan: devel_ack+ ratamir: testing_ack+

Attachments	(Terms of Use)
Add an attachment (proposed patch, testcase, etc.)

External Trackers
Tracker	ID	Priority	Status	Summary	Last Updated
oVirt gerrit	62791	master	MERGED	vdsbroker: introducing MoveStorageDomainDeviceVDSCommand	2016-12-13 11:04 EST
oVirt gerrit	62792	master	MERGED	core: Introducing MoveStorageDomainDeviceCommand	2016-12-13 11:07 EST
oVirt gerrit	62793	master	MERGED	core: adding RemoveDeviceFromSANStorageDomainCommand	2016-12-13 11:08 EST
oVirt gerrit	63098	master	MERGED	vdsbroker: HSMGetStorageDomainInfo - return the domain metadata devices	2016-12-13 10:47 EST
oVirt gerrit	63099	master	MERGED	core: AddSANStorageDomainCommand - update the domain metadata device	2016-12-13 11:03 EST
oVirt gerrit	63100	master	MERGED	vdsbroker: adding ReduceStorageDomainVDSCommand	2016-12-13 11:06 EST
oVirt gerrit	63101	master	MERGED	core: Introducing ReduceStorageDomainDeviceCommand	2016-12-13 11:07 EST
oVirt gerrit	63981	master	MERGED	core: AddExistingBlockStorageDomain - complete the metadata devices info	2016-12-13 11:03 EST
oVirt gerrit	67895	master	MERGED	core: adding the ReduceDeviceFromStorageDomain config value	2016-12-13 11:03 EST
oVirt gerrit	67896	master	MERGED	core: StorageJob - adding support to inspect the job error code	2016-12-13 11:04 EST
oVirt gerrit	67897	master	MERGED	core: introducing ReduceSANStorageDomainDevicesCommand	2016-12-13 11:08 EST
oVirt gerrit	67898	master	MERGED	api: adding support to reduce luns from domain	2016-12-13 11:09 EST
oVirt gerrit	72665	master	MERGED	lvm: enabling movePV()	2017-02-22 14:36 EST
oVirt gerrit	72978	ovirt-4.1	MERGED	lvm: enabling movePV()	2017-02-28 06:14 EST

Groups: None (edit)

Description Tal Nisan 2016-12-12 08:09:55 EST

Description of problem:
Add the ability to remove LUN(s) from an existing storage domain

Comment 1 Liron Aravot 2016-12-28 02:27:53 EST

Allon/Tal, this feature was completely merged but its currently disabled until the issue described in 1403836 will be solved.
How do we want to mark it in bugzilla?

Comment 2 Liron Aravot 2016-12-29 04:54:31 EST

After talk with ydary, marking as BLOCKED till the lvmetad issue this bug depends on will be solved either by us (disabling lvmetad) or in lvm.

Comment 3 Tal Nisan 2017-01-12 08:23:40 EST

Liron, please fill out a doc text.
As for the other question, I've asked the QE guys to verify this feature with manually disabling lvmetad so they are not blocked on this bug for now

Comment 4 Lilach Zitnitski 2017-03-09 04:00:43 EST

--------------------------------------
Tested with the following code:
----------------------------------------
rhevm-4.1.1.3-0.1.el7.noarch
vdsm-4.19.7-1.el7ev.x86_64

Tested with the following scenario:

Steps to Reproduce:
1. move the storage domain to maintenance 
2. using REST API reduce the lun using it's lun id
3. activate the storage domain

Actual results:
storage domain is reduced and the total space is updated

Expected results:

Moving to VERIFIED!

Comment 5 Marina 2018-02-15 10:05:59 EST

https://www.ovirt.org/develop/release-management/features/storage/reduce-luns-from-sd/

Note You need to log in before you can comment on or make changes to this bug.