Saggi Mizrahi has posted comments on this change.

Change subject: utils: add monotonic_time function
......................................................................


Patch Set 3: Code-Review-1

(1 comment)

http://gerrit.ovirt.org/#/c/35350/3/lib/vdsm/utils.py
File lib/vdsm/utils.py:

Line 1215:       found resolution is 10 (ten) ms.
Line 1216:     * This functions is resilient with respect to system clock
Line 1217:       adjustments.
Line 1218:     """
Line 1219:     return os.times()[-1]
The proper way is os.times()[4] as the amount of items can change between 
versions and OSs.


-- 
To view, visit http://gerrit.ovirt.org/35350
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I65c7d6041ed89fc8c497115b2d70341e82936b26
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com>
Gerrit-Reviewer: Francesco Romani <from...@redhat.com>
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizr...@redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to