Closed
Bug 1345350
Opened 8 years ago
Closed 8 years ago
Intermittent toolkit/components/windowcreator/test/test_private_window_from_content.html | any mochitest-32 | application ran for longer than allowed maximum time
Categories
(Firefox for Android Graveyard :: Testing, defect, P1)
Firefox for Android Graveyard
Testing
Tracking
(firefox-esr52 unaffected, firefox53 unaffected, firefox54 unaffected, firefox55 fixed)
RESOLVED
FIXED
Firefox 55
Tracking | Status | |
---|---|---|
firefox-esr52 | --- | unaffected |
firefox53 | --- | unaffected |
firefox54 | --- | unaffected |
firefox55 | --- | fixed |
People
(Reporter: intermittent-bug-filer, Assigned: gbrown)
References
Details
(Keywords: intermittent-failure, Whiteboard: [stockwell fixed])
Attachments
(1 file)
(deleted),
patch
|
jmaher
:
review+
|
Details | Diff | Splinter Review |
Assignee | ||
Comment 1•8 years ago
|
||
Screenshot as in bug 1336541 - related?
Also notice that the first run dialog for tab queues is displayed...that seems unexpected.
Assignee | ||
Comment 2•8 years ago
|
||
The job here is Android Debug mochitest-32. This is perma-failing on autoland starting with https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=4dc0edd4681c71fb0e67bc19bbc290616952fdf7&filter-searchStr=mochitest-32...but it was running long (90-100 minutes) in earlier pushes.
There was a significant increase in run time a few days earlier, at https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&filter-searchStr=mochitest-32&tochange=a793136c90bc5d32f2c82aa7dea4bc300c4f1836&fromchange=31c09bb63b697089f9866b7cbfa57b8900df2b1d.
Assignee: nobody → gbrown
Priority: -- → P1
Assignee | ||
Comment 3•8 years ago
|
||
Most of the Android 4.3 Debug mochitest jobs run in 40 to 60 minutes, but mochitest-31 and mochitest-32 are closer to 90 minutes.
Assignee | ||
Comment 4•8 years ago
|
||
This looks to be a regression from bug 1335751: at least some Android tests started running a little slower with that landing:
https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&filter-searchStr=android%20mochitest-32&tochange=c10963d3a6872149f753738c265ef56b4394dda0&fromchange=340e7f8a7229786fc8808d4a78983338997f81f3
There was an earlier landing (later backed out) with the same result:
https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&tochange=3c4a6d36149c38695e0cdccd9e7bbe261d1e17c8&fromchange=8ab76e34613bf4af022e05d14ed5f75529ed951d&filter-searchStr=mochitest-32
I don't think there's anything "wrong" with bug 1335751, but let's just check in there. :jonco - Just wanted to make you aware that some Android mochitests run slower with your patches; if that is surprising or concerning to you, let me know.
Probably the best action here is to add more test chunks so we have the capacity to run longer.
Assignee | ||
Comment 5•8 years ago
|
||
Assignee | ||
Updated•8 years ago
|
Summary: Intermittent toolkit/components/windowcreator/test/test_private_window_from_content.html | application ran for longer than allowed maximum time → Intermittent toolkit/components/windowcreator/test/test_private_window_from_content.html | any mochitest-32 | application ran for longer than allowed maximum time
Assignee | ||
Comment 7•8 years ago
|
||
Assignee | ||
Comment 8•8 years ago
|
||
At 40 chunks there are still several chunks taking longer than 70 minutes.
48 chunks seems to bring almost all jobs into the 30 - 60 minute range.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=97798692b4534acc5f68fc10d0e300790edb4bf5
Attachment #8844988 -
Flags: review?(jmaher)
Comment 9•8 years ago
|
||
Comment on attachment 8844988 [details] [diff] [review]
increase android debug mochitest chunks to 48
Review of attachment 8844988 [details] [diff] [review]:
-----------------------------------------------------------------
insane, but needed.
Attachment #8844988 -
Flags: review?(jmaher) → review+
Comment 10•8 years ago
|
||
Pushed by gbrown@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ec475e2b1915
Increase Android debug mochitest chunks; r=jmaher
Looks like it worked. No chunks exceeded the max runtime.
Comment 12•8 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 55
Comment 13•8 years ago
|
||
(In reply to Geoff Brown [:gbrown] from comment #4)
> I don't think there's anything "wrong" with bug 1335751, but let's just
> check in there. :jonco - Just wanted to make you aware that some Android
> mochitests run slower with your patches; if that is surprising or concerning
> to you, let me know.
(jonco is on PTO) Yes, it added some slower but more complete gray marking checks.
Flags: needinfo?(jcoppeard)
Updated•8 years ago
|
status-firefox53:
--- → unaffected
status-firefox54:
--- → unaffected
status-firefox-esr52:
--- → unaffected
Comment hidden (Intermittent Failures Robot) |
Updated•8 years ago
|
Whiteboard: [stockwell fixed]
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
•