Closed
Bug 1390011
Opened 7 years ago
Closed 7 years ago
browser_getScreenshots.js is going to intermittently fail on Linux when Gecko 57 merges to Beta on 2017-09-20
Categories
(Firefox :: New Tab Page, defect)
Tracking
()
VERIFIED
FIXED
Firefox 57
Tracking | Status | |
---|---|---|
firefox57 | --- | verified |
People
(Reporter: RyanVM, Assigned: Mardak)
References
Details
Only appears to affect Linux32 debug, for reasons I can't explain beyond general "slow as molasses" reasons. Probably not worth spending much time on beyond skipping the test, but worth filing in case someone else has any insights.
https://treeherder.mozilla.org/logviewer.html#?job_id=122807510&repo=try
TEST-UNEXPECTED-FAIL | browser/extensions/activity-stream/test/functional/mochitest/browser_getScreenshots.js | Test timed out
Assignee | ||
Comment 1•7 years ago
|
||
Was the original test disabled for various platforms/builds before? We might want to match some of that (except for the one caused by GCing the observer)
Flags: needinfo?(usarracini)
Comment 2•7 years ago
|
||
From what I remember, the original test (browser_PreviewProvider.js) was disabled on both linux and windows, both for frequent time outs (which I guess is the GCing the observer thing - bug 1343150)
Flags: needinfo?(usarracini)
Reporter | ||
Comment 3•7 years ago
|
||
Latest set of Try pushes suggests this is just a frequent intermittent on Linux builds in general.
Summary: browser_getScreenshots.js is going to permafail on Linux32 debug when Gecko 57 merges to Beta on 2017-09-20 → browser_getScreenshots.js is going to intermittently fail on Linux when Gecko 57 merges to Beta on 2017-09-20
Reporter | ||
Comment 4•7 years ago
|
||
Looks like this test got disabled across the board by bug 1389482. Not sure if we should bother leaving this open at this point.
Flags: needinfo?(edilee)
Assignee | ||
Comment 5•7 years ago
|
||
k88hudson's turning off the test was somewhat accidental as part of how we were exporting beta changes to central for uplift. She also explicitly kept it disabled when exporting with bug 1391336.
I pushed some dumping to try to see where it's timing out:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=79a3da54fdf104853dc15af6ab72c9b4856f466f
Looking through bug 1343150 where this test came from, it looks like it was only intermittent on linux for a week per bug 1343150 comment 1 but then spiked from the GC weak reference change per bug 1343150 comment 3. We turned on the test after fixing the GC weak reference issue, so maybe it is indeed a linux specific slowness.
Assignee | ||
Comment 6•7 years ago
|
||
(In reply to Ed Lee :Mardak from comment #5)
> https://treeherder.mozilla.org/#/jobs?repo=try&revision=79a3da54fdf104853dc15af6ab72c9b4856f466f
Looks to be failing about 13% of the time on Linux32/64 debug or Stylo opt. But even on Linux32 debug, the test is normally finishing within 8 seconds max.
Maybe we'll just skip the test on linux for now.
I'll do a little bit more debugging as there might be an underlying failure to capture.. https://treeherder.mozilla.org/#/jobs?repo=try&revision=2c9a51970b59630da2fea9155097906c479eb2b2
Assignee | ||
Comment 7•7 years ago
|
||
ursula: Looks like it just needed to await the pref change:
https://hg.mozilla.org/try/rev/69adbb8c4c76bf861edeef3a19589e545489ecc2#l3.41
https://treeherder.mozilla.org/#/jobs?repo=try&revision=69adbb8c4c76bf861edeef3a19589e545489ecc2
Flags: needinfo?(edilee)
Assignee | ||
Comment 8•7 years ago
|
||
Fixed by https://github.com/mozilla/activity-stream/pull/3212 to be uplifted via bug 1392384.
Assignee: nobody → edilee
Blocks: 1392384
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 9•7 years ago
|
||
I would prefer that we leave this open until the fix actually lands on mozilla-central.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 10•7 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 7 years ago → 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 57
Reporter | ||
Comment 11•7 years ago
|
||
Looks good AFAICT. I'll reopen if I see them again.
Status: RESOLVED → VERIFIED
Updated•5 years ago
|
Component: Activity Streams: Newtab → New Tab Page
You need to log in
before you can comment on or make changes to this bug.
Description
•