Ayal Baron has posted comments on this change.
Change subject: Fix rollback after failure to create snapshot
......................................................................
Patch Set 4: Code-Review-1
(1 comment)
....................................................
File vdsm/storage/volume.py
Line 329: pvolUUID)
Line 330: pvol.prepare()
Line 331: try:
Line 332: if not pvol.recheckIfLeaf():
Line 333: pvol.setLeaf()
again, if recheckIfLeaf returns False then it is not safe to call it here and
if it is returning true then there is nothing left to do.
For example, in this flow you would change a shared volume into a leaf (i.e. a
template into a leaf).
Line 334: except Exception:
Line 335: cls.log.error("Unexpected error", exc_info=True)
Line 336: finally:
Line 337: pvol.teardown(sdUUID, pvolUUID)
--
To view, visit http://gerrit.ovirt.org/18205
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I11948aacad21008ffbb4cb23013bf363e60bdb9b
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Sergey Gotliv <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Eduardo <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Sergey Gotliv <[email protected]>
Gerrit-Reviewer: Yeela Kaplan <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
