Closed
Bug 1401022
Opened 7 years ago
Closed 6 years ago
Android debug mochitest, reftest max-run-time is excessive
Categories
(Firefox for Android Graveyard :: Testing, enhancement, P5)
Firefox for Android Graveyard
Testing
Tracking
(firefox57 affected)
RESOLVED
WONTFIX
Tracking | Status | |
---|---|---|
firefox57 | --- | affected |
People
(Reporter: gbrown, Assigned: gbrown)
References
Details
Attachments
(1 file)
858 bytes,
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
The test task max-run-time parameter for Android Debug mochitests and reftests is an astounding 10800 seconds - much more than required. This may have been accidentally introduced in the buildbot -> taskcluster transition.
I think 5400 would be okay for everything but perhaps Android Debug mochitest-48, but let's try 7200 first.
(10800 means that we are waiting up to 3 hours for results on hung tasks, while most of these tasks normally complete in 30 to 60 minutes.)
Assignee | ||
Comment 1•7 years ago
|
||
Complication: looks like the application timeout for debug mochitest is 5400 and debug reftest is 7200.
Assignee | ||
Comment 2•7 years ago
|
||
(In reply to Geoff Brown [:gbrown] from comment #1)
> Complication: looks like the application timeout for debug mochitest is 5400
> and debug reftest is 7200.
No, it's 5400 for all Android opt and debug tests. Sometimes it takes a few minutes longer to hit the application timeout ... bug 1401287? or inexact implementation??
Assignee | ||
Comment 3•7 years ago
|
||
Seems OK:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=9539e8c56e72ed69b49e202cdddf173403b19d01
I'm not done here. I'll leave-open, monitor, see if this can be reduced further in a week or so.
Attachment #8909931 -
Flags: review?(jmaher)
Updated•7 years ago
|
Attachment #8909931 -
Flags: review?(jmaher) → review+
Assignee | ||
Updated•7 years ago
|
Keywords: leave-open
Pushed by gbrown@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/7c2ee4964472
Reduce taskcluster max-run-time for Android Debug mochitests and reftests; r=jmaher
Comment 5•7 years ago
|
||
bugherder |
Assignee | ||
Updated•7 years ago
|
Priority: -- → P1
Assignee | ||
Comment 6•7 years ago
|
||
This remains a concern but I'm not making progress. As more tests are run, more and more run time is required. What's worse is that some chunks are much slower than others. We can address that with more test chunks, but that adds some overhead and aws $. For now we run long chunks and keep max run time relatively large.
Will moving to geckoview-based testing help? Will we move to x86 kvm-accelerated emulators? Let's wait and see what efficiency improvements the future holds.
Assignee: gbrown → nobody
Priority: P1 → P3
Comment 7•6 years ago
|
||
Re-triaging per https://bugzilla.mozilla.org/show_bug.cgi?id=1473195
Needinfo :susheel if you think this bug should be re-triaged.
Priority: P3 → P5
Comment hidden (Intermittent Failures Robot) |
Comment 9•6 years ago
|
||
The leave-open keyword is there and there is no activity for 6 months.
:gbrown, maybe it's time to close this bug?
Flags: needinfo?(gbrown)
Assignee | ||
Comment 10•6 years ago
|
||
Sure. The max-run-time remains excessive but cannot be reduced without adding an unreasonable number of chunks. Eventually we will retire the android-em-4.3 tests.
Assignee: nobody → gbrown
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(gbrown)
Keywords: leave-open
Resolution: --- → WONTFIX
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•