Closed
Bug 1343520
Opened 9 years ago
Closed 9 years ago
Intermittent test_direct_update.py TestDirectUpdate.test_update | InvalidArgumentException:
Categories
(Testing :: Firefox UI Tests, defect)
Tracking
(firefox52 unaffected, firefox-esr52 unaffected, firefox53 unaffected, firefox54 fixed)
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| firefox52 | --- | unaffected |
| firefox-esr52 | --- | unaffected |
| firefox53 | --- | unaffected |
| firefox54 | --- | fixed |
People
(Reporter: intermittent-bug-filer, Assigned: whimboo)
References
Details
(Keywords: intermittent-failure, regression, Whiteboard: [stockwell fixed])
| Assignee | ||
Comment 1•9 years ago
|
||
There was a huge spike for nightly builds yesterday:
https://treeherder.mozilla.org/#/jobs?repo=mozilla-central&revision=1bc2ad020aee2830e0a7941f10958dbec108c254&filter-searchStr=Firefox+&filter-tier=1&filter-tier=2&filter-tier=3
For today it looks a bit better. But I wonder where this is coming from...
> 04:56:06 INFO - 1488372966577 Marionette TRACE conn7 -> [0,3,"setTimeouts",{"pageLoad":300000}]
> 04:56:06 INFO - 1488372966580 Marionette TRACE conn7 <- [1,3,{"error":"invalid argument","message":"","stacktrace":"WebDriverError@chrome://marionette/content/error.js:160:5\nInvalidArgumentError@chrome://marionette/content/error.js:222:5\nfromJSON@chrome://marionette/content/session.js:72:17\nGeckoDriver.prototype.setTimeouts@chrome://marionette/content/driver.js:1508:19\nDispatcher.prototype.execute/req<@chrome://marionette/content/dispatcher.js:128:20\nTaskImpl_run@resource://gre/modules/Task.jsm:319:42\nTaskImpl@resource://gre/modules/Task.jsm:277:3\nasyncFunction@resource://gre/modules/Task.jsm:252:14\nTask_spawn@resource://gre/modules/Task.jsm:166:12\nDispatcher.prototype.execute@chrome://marionette/content/dispatcher.js:118:13\nDispatcher.prototype.onPacket@chrome://marionette/content/dispatcher.js:89:5\n_onJSONObjectReady/<@chrome://marionette/content/server.js -> resource://devtools/shared/transport/transport.js:483:11\nexports.makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/ThreadSafeDevToolsUtils.js:101:14\nexports.makeInfallible/<@resource://gre/modules/commonjs/toolkit/loader.js -> resource://devtools/shared/ThreadSafeDevToolsUtils.js:101:14\n"},null]
So we fail in setting the timeout for page load correctly. And now I can remember for bug 1342162. So this is a regression visible a couple of days until funsize doesn't produce mar files anymore for builds which used the old property name. Given that the patch landed yesterday it will cause failures up to March 4th.
Blocks: 1342162
Keywords: regression
| Assignee | ||
Comment 2•9 years ago
|
||
Actually this won't go away from itself. It will hit us when we have to do update tests for releases given that the patch on bug 1342162 will never allow us to run update tests from 52.0, and which is a blocker for QA. So I will request a backout of the patch, for a sane fix.
| Assignee | ||
Comment 3•9 years ago
|
||
The patch on bug 1342162 got backed out. So this should be fixed the next days. I will leave the bug open for an easier starring of failures on Treeherder until then.
Comment 4•9 years ago
|
||
thanks :whimboo for taking care of this and tracking it.
Whiteboard: [stockwell fixed]
| Comment hidden (Intermittent Failures Robot) |
| Assignee | ||
Comment 6•9 years ago
|
||
Everything is back to normal for update tests on mozilla-central. Closing bug as fixed.
Assignee: nobody → hskupin
Status: NEW → RESOLVED
Closed: 9 years ago
status-firefox53:
--- → unaffected
status-firefox54:
--- → fixed
Resolution: --- → FIXED
Updated•9 years ago
|
status-firefox52:
--- → unaffected
status-firefox-esr52:
--- → unaffected
You need to log in
before you can comment on or make changes to this bug.
Description
•