Michal Skrivanek has posted comments on this change.
Change subject: vdsm: Delay desktop lock until we are sure it was a disconnect
......................................................................
Patch Set 2: (1 inline comment)
....................................................
File vdsm/vm.py
Line 840: # guest. And only lock it, if it there was no new connect.
Line 841: # This is detected by the clientIp being set or not.
Line 842: self._desktopLockTimer =
threading.Timer(_DESKTOP_LOCK_TIMEOUT,
Line 843:
self._timedDesktopLock)
Line 844: self._desktopLockTimer.start()
where is the onConnect invalidation of this timer?
Line 845:
Line 846: def _rtcUpdate(self, timeOffset):
Line 847: self.log.debug('new rtc offset %s', timeOffset)
Line 848: self.conf['timeOffset'] = timeOffset
--
To view, visit http://gerrit.ovirt.org/13798
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I6af5cf868dc9caaa07134b2b296d985f06cf15f2
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Vinzenz Feenstra <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Peter V. Saveliev <[email protected]>
Gerrit-Reviewer: Vinzenz Feenstra <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
