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