Royce Lv has uploaded a new change for review.
Change subject: persist state in _changeBlockDev ......................................................................
persist state in _changeBlockDev
persist state to make vm.conf survive recovery
Change-Id: I220b72ebb360356092a2dccc25dfb9f63f638607 Signed-off-by: Royce Lv lvroyce@linux.vnet.ibm.com --- M vdsm/libvirtvm.py 1 file changed, 2 insertions(+), 0 deletions(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/17/8617/1
diff --git a/vdsm/libvirtvm.py b/vdsm/libvirtvm.py index bd43b56..abd8440 100644 --- a/vdsm/libvirtvm.py +++ b/vdsm/libvirtvm.py @@ -2028,6 +2028,8 @@ ['message']}} self.cif.teardownVolumePath(self.conf.get(vmDev)) self.conf[vmDev] = path + # persist state to keep vm conf consistency in recovery + self.saveState() return {'status': doneCode, 'vmList': self.status()}
def setTicket(self, otp, seconds, connAct, params):
-- To view, visit http://gerrit.ovirt.org/8617 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange Gerrit-Change-Id: I220b72ebb360356092a2dccc25dfb9f63f638607 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Royce Lv lvroyce@linux.vnet.ibm.com
Itamar Heim has posted comments on this change.
Change subject: persist state in _changeBlockDev ......................................................................
Patch Set 1:
still relevant or should be abandoned?
Royce Lv has abandoned this change.
Change subject: persist state in _changeBlockDev ......................................................................
Abandoned
vdsm-patches@lists.fedorahosted.org