Change in vdsm[master]: formatStorageDomain removes the VG too.

danken at redhat.com danken at redhat.com
Mon Aug 8 12:51:28 UTC 2011


Dan Kenigsberg has submitted this change and it was merged.

Change subject: formatStorageDomain removes the VG too.
......................................................................


formatStorageDomain removes the VG too.

When removing a SD the VG should be removed too (atomicity).
Actually this is what RHEV-M does. Therefore the functions
split is redundant.
The removeVG function is marked as DEPRECATED (to be removed).
The VG is removed in this version in order to minimize the
chance to remain with a VG thats not belongs to a SD.

Change-Id: Ie993abd2a812f1ecec127cc637794994591e48ea
---
M vdsm/storage/blockSD.py
M vdsm/storage/hsm.py
2 files changed, 7 insertions(+), 5 deletions(-)

Approvals:
  Dan Kenigsberg: Verified; Looks good to me, approved


--
To view, visit http://gerrit.usersys.redhat.com/659
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie993abd2a812f1ecec127cc637794994591e48ea
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo Warszawski <ewarszaw at redhat.com>
Gerrit-Reviewer: Ayal Baron
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Eduardo Warszawski <ewarszaw at redhat.com>
Gerrit-Reviewer: Yotam Oron <yoron at redhat.com>


More information about the vdsm-patches mailing list