Change in vdsm[master]: Marking del/zero volumes in blockSD class.

danken at redhat.com danken at redhat.com
Sun Apr 7 09:50:24 UTC 2013


Dan Kenigsberg has submitted this change and it was merged.

Change subject: Marking del/zero volumes in blockSD class.
......................................................................


Marking del/zero volumes in blockSD class.

Previous delete was not atomic since lvremove of multiple LVs can
partially fail (e.g. since volume is open) leaving some volumes
intact and others deleted.
Marking before delete will enable garbage collection in such cases.
The old rename convention is not atomic since the rename should be
done sequentially.

Related to: BZ#905038, BZ#910013, BZ#875708

Change-Id: I747bc2218e0b2cc256dd352ad890143c7d354bd8
Signed-off-by: Eduardo <ewarszaw at redhat.com>
---
M vdsm/storage/blockSD.py
M vdsm/storage/sd.py
2 files changed, 24 insertions(+), 11 deletions(-)

Approvals:
  Ayal Baron: Looks good to me, approved
  Dan Kenigsberg: Verified


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I747bc2218e0b2cc256dd352ad890143c7d354bd8
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Daniel Paikov <paikov at gmail.com>
Gerrit-Reviewer: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Gadi Ickowicz <gickowic at redhat.com>
Gerrit-Reviewer: Haim Ateya <hateya at redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list