Change in vdsm[master]: virt: move recovery load into VmState

fromani at redhat.com fromani at redhat.com
Wed Jan 27 21:16:52 UTC 2016


Francesco Romani has posted comments on this change.

Change subject: virt: move recovery load into VmState
......................................................................


Patch Set 6:

(1 comment)

https://gerrit.ovirt.org/#/c/51389/6/vdsm/virt/recovery.py
File vdsm/virt/recovery.py:

Line 96:             if response.is_error(cif.createVm(self._restore(),
Line 97:                                               vmRecover=True)):
Line 98:                 return False
Line 99:         except:
Line 100:             self._log.exception("Error recovering VM: %s", self._vmid)
> introduce self._vmid into this patch, not in the parent; it is useless in t
Done
Line 101:             return False
Line 102:         else:
Line 103:             return True
Line 104: 


-- 
To view, visit https://gerrit.ovirt.org/51389
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Iea65e129c6016f5b75fc228eaf88492a0d10d39f
Gerrit-PatchSet: 6
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list