Closed
Bug 1247033
Opened 9 years ago
Closed 9 years ago
wpt reftests: bdi-paragraph-level-container.html is timing out more than usual
Categories
(Testing :: General, defect)
Testing
General
Tracking
(firefox47 fixed)
RESOLVED
FIXED
mozilla47
Tracking | Status | |
---|---|---|
firefox47 | --- | fixed |
People
(Reporter: armenzg, Unassigned)
References
Details
Attachments
(1 file, 1 obsolete file)
1.04 KB,
patch
|
dustin
:
review+
|
Details | Diff | Splinter Review |
It seems that bug 1215394 has become a bit more intermittent, however, under docker it has become almost permanent.
Is there a way to increase the timing out for this specific job?
I can also bump the instance size.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=933d4053fd9e&filter-searchStr=Linux%20x64%20debug%20Desktop%20web-platform-tests%20[TC]%20Linux64%20web-platform-tests-reftests%20tc-W%28Wr%29&selectedJob=16520474
https://brasstacks.mozilla.com/orangefactor/?display=Bug&bugid=1215394&startday=2016-01-21&endday=2016-02-09&tree=all
Reporter | ||
Comment 1•9 years ago
|
||
Reporter | ||
Comment 2•9 years ago
|
||
Attachment #8717641 -
Attachment is obsolete: true
Reporter | ||
Comment 3•9 years ago
|
||
jgraham: besides the review request, could you please have a look to comment 0?
Reporter | ||
Updated•9 years ago
|
Attachment #8718045 -
Flags: review?(james)
Comment 4•9 years ago
|
||
For one specific test you add <meta name=timeout content=long> to the test to get a longer timeout (60s rather than 10s by default although there's an additional multiplier that means those specific values probably aren't what's actually being used).
In general I am pretty surprised if this test is timing out, ever, so it would be super-interesting if you could use the fact that it's always timing out to find out what's actually going on here.
I don't really feel like I'm the right person to review the patch to up the instance size since I know ~nothing about TC but consider it f+ on the idea if we can first do some investigation of the underlying issue.
Reporter | ||
Comment 5•9 years ago
|
||
It is *almost* permanent. I will try to reproduce it locally.
Reporter | ||
Updated•9 years ago
|
Attachment #8718045 -
Flags: review?(james)
Reporter | ||
Comment 6•9 years ago
|
||
Unfortunately there is a lot of intermittent jobs:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=1dca45d3b4dd&filter-searchStr=web-platform%20reftest
I'm going to bump the instance size and leave this open until we figure out why.
Reporter | ||
Comment 7•9 years ago
|
||
Comment on attachment 8718045 [details] [diff] [review]
Run wpt reftests on larger instance (same as other wpt tests)
For now.
Attachment #8718045 -
Flags: review?(dustin)
Reporter | ||
Comment 8•9 years ago
|
||
Reporter | ||
Comment 9•9 years ago
|
||
Updated•9 years ago
|
Attachment #8718045 -
Flags: review?(dustin) → review+
Reporter | ||
Comment 10•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/9227f129d064ae2f022fc7732f4bf4055f3c9207
Bug 1247033 - Run wpt reftests in larger instance. DONTBUILD. r=dustin
Reporter | ||
Comment 11•9 years ago
|
||
Reporter | ||
Comment 12•9 years ago
|
||
Reporter | ||
Comment 13•9 years ago
|
||
Reporter | ||
Comment 14•9 years ago
|
||
Comment 15•9 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox47:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
You need to log in
before you can comment on or make changes to this bug.
Description
•