Yaniv Bronhaim 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 <[email protected]>
Change-Id: I4a6fb01775f7c834a3466e45b9b7ed9b4bd5c3be
Reviewed-on: http://gerrit.ovirt.org/29303
Reviewed-by: Nir Soffer <[email protected]>
Reviewed-by: Dan Kenigsberg <[email protected]>
Reviewed-by: Yaniv Bronhaim <[email protected]>
---
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
  Yaniv Bronhaim: Looks good to me, approved
  Dan Kenigsberg: Looks good to me, but someone else must approve
  Maor Lipchuk: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4a6fb01775f7c834a3466e45b9b7ed9b4bd5c3be
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: [email protected]
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to