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 <[email protected]> --- 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 <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Daniel Paikov <[email protected]> Gerrit-Reviewer: Eduardo <[email protected]> Gerrit-Reviewer: Gadi Ickowicz <[email protected]> Gerrit-Reviewer: Haim Ateya <[email protected]> Gerrit-Reviewer: Yeela Kaplan <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
