Dan Kenigsberg has posted comments on this change.
Change subject: tests: Add time per test if nose-timer is available
......................................................................
Patch Set 1:
(1 comment)
http://gerrit.ovirt.org/#/c/30171/1/tests/testrunner.py
File tests/testrunner.py:
Line 48: timer_plugin = TimerPlugin()
Line 49: timer_plugin.enabled = True
Line 50: timer_plugin.timer_ok = 500 # Okay <= 500ms
Line 51: timer_plugin.timer_warning = 5000 # Warn > 5s
Line 52: timer_plugin.timer_no_color = False # Enable color output
These defaults are not correct for all tests.
these numbers are
indeed a bit arbitrary, but the alternative of tweaking the plugin and running all over
our tests defining acceptable times do not sound appealing, either.
Can we drop colours instead?
Line 53:
Line 54: from testValidation import SlowTestsPlugin, StressTestsPlugin
Line 55:
Line 56: import zombiereaper
--
To view, visit
http://gerrit.ovirt.org/30171
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4960b30532a84abd259fb268c7e40a1751847a96
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <asegurap(a)redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes