Decrease telemetry-tests-client CI max-run-time value
Categories
(Toolkit :: Telemetry, enhancement, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox67 | --- | fixed |
People
(Reporter: raphael, Assigned: grumpy)
References
Details
Attachments
(1 file)
:aryx pointed out that telemetry-tests-client CI jobs typically complete in less than 10 minutes. That is why it is sensible to update the "max-run-time" setting, so that hanging test runs abort early rather than blocking for a long time.
Update "max-run-time" to "600".
https://searchfox.org/mozilla-central/source/taskcluster/ci/test/misc.yml#57
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
:raphael
Is there a way to verfiy that this change works when running them locally?
Assignee | ||
Updated•5 years ago
|
Reporter | ||
Updated•5 years ago
|
Reporter | ||
Comment 2•5 years ago
|
||
I don't think the setting has any effect on running tests locally.
You could however add a Python "time.sleep()" into any of the telemetry-tests-client tests with parameter longer than the new max-run-time and see what happens. Adding :aryx for ni on whether this is a good idea.
Comment 3•5 years ago
|
||
The time limits are not taken into account when run locally because the machine could be slower than CI. That the limit gets enforced is shown regularly in production (hangs, slow cloning etc.). No need to test in my opinion. If you want you can test it with Try.
Assignee | ||
Comment 4•5 years ago
|
||
Pushed by rpierzina@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/e57c2f2ddeff Decrease telemetry-tests-client CI max-run-time value; r=raphael
Comment 6•5 years ago
|
||
bugherder |
Reporter | ||
Updated•5 years ago
|
Description
•