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)

Version 3
defect
Not set
normal

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])

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
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.
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.
thanks :whimboo for taking care of this and tracking it.
Whiteboard: [stockwell fixed]
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
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.