Change in vdsm[master]: BZ#744704 - Fix rollback support for complex tasks
danken at redhat.com
danken at redhat.com
Thu Nov 10 15:56:56 UTC 2011
Dan Kenigsberg has posted comments on this change.
Change subject: BZ#744704 - Fix rollback support for complex tasks
......................................................................
Patch Set 4: (1 inline comment)
....................................................
File vdsm/storage/blockVolume.py
Line 185: else:
Line 186: # should stay %d and size should be int(size)
Line 187: volsize = "%s" % (size / 2 / 1024)
Line 188:
Line 189: # Stub rollback, just to mark the start of the task
this is why it is called nit-picking!
Line 190: vars.task.pushRecovery(task.Recovery(task.SENTINEL_ROLLBACK, "blockVolume", "BlockVolume", "startCreateVolumeRollback",
Line 191: [sdUUID, imgUUID, volUUID]))
Line 192: # create volume rollback
Line 193: vars.task.pushRecovery(task.Recovery("halfbaked volume rollback", "blockVolume", "BlockVolume", "halfbakedVolumeRollback",
--
To view, visit http://gerrit.usersys.redhat.com/1041
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie36ac02475f3f6438a63fbcbd26730862216cebd
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Ayal Baron
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
More information about the vdsm-patches
mailing list