Change in vdsm[master]: virt: introduce support for periodic operations
danken at redhat.com
danken at redhat.com
Mon Mar 30 21:38:04 UTC 2015
Dan Kenigsberg has posted comments on this change.
Change subject: virt: introduce support for periodic operations
......................................................................
Patch Set 29: Code-Review-1
(1 comment)
-1 for visibility
https://gerrit.ovirt.org/#/c/37593/29/tests/periodicTests.py
File tests/periodicTests.py:
Line 144: def _work():
Line 145: invokations[0] += 1
Line 146: invokations[1] = monotonic_time()
Line 147: if invokations[0] == BLOCK_AT:
Line 148: # must be > (PERIOD * TIMES) ~= forever
may I ask why you opted against an explicit done.wait() in this thread?
Line 149: time.sleep(10 * PERIOD * TIMES)
Line 150: executions[0] += 1
Line 151: executions[1] = monotonic_time()
Line 152: if invokations[0] == TIMES:
--
To view, visit https://gerrit.ovirt.org/37593
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I834c067b1d3dba16a1f8c83e555d2cc04cbf32ed
Gerrit-PatchSet: 29
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes
More information about the vdsm-patches
mailing list