Closed Bug 1101770 Opened 5 years ago Closed 5 years ago

[steeplechase] add over all timeout

Categories

(Core :: WebRTC, defect)

36 Branch
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla36

People

(Reporter: drno, Assigned: drno)

Details

Attachments

(1 file)

As the 300s timeout for mochitest does not work if tests get executed in steeplechase we need another high level timeout to prevent the test machines getting filled/blocked with open browser windows in case a test fails.
This adds one high level timer in case a test gets executed in steeplechase which exits the whole test if it has not finished by then.

Try run: https://treeherder.mozilla.org/ui/#/jobs?repo=try&revision=19ef23fc91e1
Attachment #8525591 - Flags: review?(spolk)
Comment on attachment 8525591 [details] [diff] [review]
bug_1101770_add_steeplechase_timeout.patch

Review of attachment 8525591 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good, as long as the timeout is 30s and not 300s.
Attachment #8525591 - Flags: review?(spolk) → review+
Manually double checked it is 30s.
BTW I ran this with an artificial error in our lab and it was exiting the hanging browser (and the other side gave up).
Try run looks green, besides some unrelated oranges.
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/9c7a7f4ca6ce
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla36
You need to log in before you can comment on or make changes to this bug.