Closed
Bug 1456400
Opened 7 years ago
Closed 7 years ago
Intermittent TEST-UNEXPECTED-TIMEOUT | /webdriver/tests/close_window/user_prompts.py | expected OK
Categories
(Remote Protocol :: Marionette, defect, P5)
Tracking
(firefox61 fixed)
RESOLVED
FIXED
mozilla61
Tracking | Status | |
---|---|---|
firefox61 | --- | fixed |
People
(Reporter: intermittent-bug-filer, Assigned: janv)
References
Details
(Keywords: intermittent-failure, Whiteboard: [retriggered][wptsync upstream])
Attachments
(1 file)
Filed by: shindli [at] mozilla.com
https://treeherder.mozilla.org/logviewer.html#?job_id=175271063&repo=autoland
https://queue.taskcluster.net/v1/task/U34zpUZgTaK_KmRq3ojz6g/runs/0/artifacts/public/logs/live_backing.log
06:00:20 INFO - Buffered messages logged at 05:55:21
06:00:20 INFO - 57 INFO got stalled event
06:00:20 INFO - Buffered messages finished
06:00:20 ERROR - 58 INFO TEST-UNEXPECTED-FAIL | dom/media/mediasource/test/test_FrameSelection_mp4.html | Test timed out.
06:00:20 INFO - reportError@SimpleTest/TestRunner.js:121:7
06:00:20 INFO - TestRunner._checkForHangs@SimpleTest/TestRunner.js:142:7
Comment 1•7 years ago
|
||
The reference in comment 0 is wrong. So here an update which shows that the failure is actually again a shutdown hang of Firefox:
[task 2018-04-24T07:00:31.762Z] 07:00:31 INFO - PID 1194 | 1524553231753
Marionette TRACE 0 -> [0,12,"Marionette:Quit",{"flags":["eForceQuit"]}]
[task 2018-04-24T07:00:31.763Z] 07:00:31 INFO - PID 1194 | 1524553231754
Marionette TRACE 0 <- [1,12,{"error":"invalid session
id","message":"","stacktrace":"WebDriverError@chrome://marionette/content/error.js:178:5\nInv
...
et@chrome://marionette/content/server.js:245:8\n_onJSONObjectReady/<@chrome://marionette/content/transport.js:500:9\n"},null]
[task 2018-04-24T07:00:31.764Z] 07:00:31 INFO - PID 1194 | 1524553231754
geckodriver::marionette TRACE <- [1,12,{"error":"invalid session
id","message":"","stacktrace":"WebDriverError@chrome://marionette/content/error.js:178:5\nInvalidSessionIDError@chrome://marionette/content/error.js:347:5\nassert.that/<@chrome://marionette/content/assert.js:405:13\nassert.session@chrome://marionette/content/assert.js:79:3\ndespatch@chrome://marionette/content/server.js:294:7\nasync*execute@chrome://marionette/content/server.js:271:11\nasync*onPacket/<@chrome://marionette/content/server.js:246:15\nasync*onPacket@chrome://marionette/content/server.js:245:8\n_onJSONObjectReady/<@chrome://marionette/content/transport.js:500:9\n"},null]
[task 2018-04-24T07:00:31.782Z] 07:00:31 INFO - PID 1194 | 1524553231771
Marionette DEBUG Closed connection 0
[task 2018-04-24T07:00:57.853Z] 07:00:57 INFO - TEST-UNEXPECTED-TIMEOUT |
/webdriver/tests/close_window/user_prompts.py | expected OK
So nothing we can do here right now. If it fails only intermittently we can
land, otherwise we might have to temporarily mark it as disabled.
Comment hidden (Intermittent Failures Robot) |
Comment 3•7 years ago
|
||
Started to retrigger jobs for this failure:
https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=295fd2df534f6a5f59f9d620be955a3e70ade74b&filter-searchStr=linux32%20opt-web-platform-tests-wdspec-e10s
on this interval:
https://treeherder.mozilla.org/#/jobs?repo=autoland&filter-searchStr=linux32%20opt-web-platform-tests-wdspec-e10s&tochange=295fd2df534f6a5f59f9d620be955a3e70ade74b&fromchange=4cc04fbbd1d7d6a4af9dbb0cbea52905cff3fef7&selectedJob=175271063
and end up with this push being the culprit:
https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=295fd2df534f6a5f59f9d620be955a3e70ade74b&filter-searchStr=linux32%20opt-web-platform-tests-wdspec-e10s
It failed 4 out 6 times and on the retriggered jobs it failed 0 times.
Comment 4•7 years ago
|
||
This is a new test as added by that push, right. Given that it fails that much, and we do not see a crash report due to the shutdown hang of Firefox, a good idea would be to increase the timeout of the test. At least then we might have a chance to see a crash as caused by the hang reporter.
Flags: needinfo?(hskupin)
Comment 5•7 years ago
|
||
Given that it continuous to have a high failure rate I will come up with a patch later today which requests a longer timeout for this test.
Comment 6•7 years ago
|
||
Comment 7•7 years ago
|
||
I missed to update the manifest. Here another try:
https://treeherder.mozilla.org/#/jobs?repo=try&revision=cda6069e1715042cb74254f04292a8deb51bcec4
Comment hidden (Intermittent Failures Robot) |
Comment hidden (mozreview-request) |
Comment 10•7 years ago
|
||
The try build looks fine and I'm going to increase the timeout now.
Comment hidden (mozreview-request) |
Updated•7 years ago
|
Attachment #8971547 -
Flags: review?(hskupin)
Comment 12•7 years ago
|
||
mozreview-review |
Comment on attachment 8971547 [details]
Bug 1456400 - [wdspec] Increase timeout for close_window/user_prompts.py test.
https://reviewboard.mozilla.org/r/240300/#review246098
Attachment #8971547 -
Flags: review?(hskupin) → review+
Comment 13•7 years ago
|
||
We have to leave this bug open until bug 1449538 has been fixed.
Keywords: leave-open
Comment 14•7 years ago
|
||
Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b27cb10f8110
[wdspec] Increase timeout for close_window/user_prompts.py test. r=whimboo
Whiteboard: [retriggered] → [retriggered][wptsync upstream error]
Comment 15•7 years ago
|
||
bugherder |
Comment hidden (Intermittent Failures Robot) |
Comment 17•7 years ago
|
||
Fixed by bug 1449538. I will keep the longer timeout present for now.
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox61:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
Updated•7 years ago
|
Assignee: nobody → hskupin
Keywords: leave-open
Comment 18•7 years ago
|
||
It was actually Jan who fixed the underlying shutdown hang in QuotaManager.
Assignee: hskupin → jvarga
Created web-platform-tests PR https://github.com/w3c/web-platform-tests/pull/10941 for changes under testing/web-platform/tests
Whiteboard: [retriggered][wptsync upstream error] → [retriggered][wptsync upstream]
Upstream PR merged
Comment 21•2 years ago
|
||
Moving bug to Remote Protocol::Marionette component per bug 1815831.
Component: geckodriver → Marionette
Product: Testing → Remote Protocol
You need to log in
before you can comment on or make changes to this bug.
Description
•