Closed Bug 1531356 Opened 5 years ago Closed 5 years ago

Decrease telemetry-tests-client CI max-run-time value

Categories

(Toolkit :: Telemetry, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
mozilla67
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

Blocks: 1531369
Assignee: nobody → chartjes

:raphael

Is there a way to verfiy that this change works when running them locally?

Flags: needinfo?(rpierzina)
Flags: needinfo?(rpierzina)
Priority: -- → P1

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.

Flags: needinfo?(aryx.bugmail)

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.

Flags: needinfo?(aryx.bugmail)
Pushed by rpierzina@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e57c2f2ddeff
Decrease telemetry-tests-client CI max-run-time value; r=raphael
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67
Type: defect → enhancement
See Also: → 1550634
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: