Change in vdsm[master]: virt: vm: Update time on VM after resume

mzamazal at redhat.com mzamazal at redhat.com
Mon Nov 23 15:39:47 UTC 2015


Milan Zamazal has posted comments on this change.

Change subject: virt: vm: Update time on VM after resume
......................................................................


Patch Set 2:

(1 comment)

Jenkins happy after rebase.

https://gerrit.ovirt.org/#/c/48860/2/vdsm/virt/vm.py
File vdsm/virt/vm.py:

Line 1208:             self.log.exception("Failed to set time")
Line 1209:             result = unicode(e)
Line 1210:         else:
Line 1211:             self.log.debug('Time updated to: %d.%09d', seconds, nseconds)
Line 1212:         return result
> (continuing discussion from patchset 1) I think we don't need this
OK, removed.
Line 1213: 
Line 1214:     def shutdown(self, delay, message, reboot, timeout, force):
Line 1215:         if self.lastStatus == vmstatus.DOWN:
Line 1216:             return response.error('noVM')


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

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


More information about the vdsm-patches mailing list