Change in vdsm[master]: detach: Support force detach on Storage Domains with old pools.

danken at redhat.com danken at redhat.com
Thu Jun 26 16:32:16 UTC 2014


Dan Kenigsberg has submitted this change and it was merged.

Change subject: detach: Support force detach on Storage Domains with old pools.
......................................................................


detach: Support force detach on Storage Domains with old pools.

Force detach of storage domain only being supported on storage domains
which are part of pool.  For supporting import Data Storage Domain
feature, we should also support force detach of Storage Domain which are
not connected to any pool.

The force detach will clear the SP_UUID meta data of the imported
Stroage Dimain, when a BLANK_UUID will be passed from the client
and will enable the engine to attach it to other DC in ifferent setups.

Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
Change-Id: I4a6fb01775f7c834a3466e45b9b7ed9b4bd5c3be
Reviewed-on: http://gerrit.ovirt.org/29042
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-by: Federico Simoncelli <fsimonce at redhat.com>
---
M vdsm/storage/hsm.py
M vdsm/storage/sd.py
2 files changed, 35 insertions(+), 5 deletions(-)

Approvals:
  Nir Soffer: Looks good to me, but someone else must approve
  Federico Simoncelli: Looks good to me, approved
  Maor Lipchuk: Verified



-- 
To view, visit http://gerrit.ovirt.org/29042
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I4a6fb01775f7c834a3466e45b9b7ed9b4bd5c3be
Gerrit-PatchSet: 15
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list