Closed Bug 1500141 Opened 2 years ago Closed 2 years ago
Perma dt in devtools/client/framework/test/browser
_toolbox _telemetry _activate _splitconsole .js when Gecko 64 merges to Beta on 2018-10-15
46 bytes, text/x-phabricator-request
|Details | Review|
[Tracking Requested - why for this release]: Central as Beta: https://treeherder.mozilla.org/#/jobs?repo=try&resultStatus=pending%2Crunning%2Ctestfailed%2Cbusted%2Cexception%2Cusercancel%2Crunnable&classifiedState=unclassified&revision=4a7b8f464994b17a81710909331fff2f1d3626a0&selectedJob=206348106 Failure log: https://treeherder.mozilla.org/logviewer.html#?job_id=206348106&repo=try&lineNumber=9336
Soledad, with Alexandre on PTO, who can fix this - it permafails on beta? Thank you in advance.
The test passes fine, and when it attempts to end, a pending unresolved promise is detected and that fails the test. The promise corresponds to an actor request: Connection closed, pending request to server1.conn76.child1/highlighter28, type showBoxModel failed This is a frequent test failure for DevTools that usually means we need to change the test so it waits for various things to be done. Now, why does it perma fail when merging to beta, I don't really understand. Marking as P1 because of the nightly to beta merge coming up.
Priority: -- → P1
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
This one is most likely related to Bug 1499483, where we made the Layout tab the default tab for the inspector sidebar. However it is not clear why it fails on Beta. Note that it is not a permafail, just a very frequent intermittent. For instance it didn't fail on `Linux x64 opt`
According to two try pushes, forcing the computed view tab instead of the layout tab fixes this particular error. I think we should go with that for the time being to unblock the situation. try without fix: https://treeherder.mozilla.org/#/jobs?repo=try&revision=f952d4ebe686227b7305f56ce9c848332a5e5248 try with fix: https://treeherder.mozilla.org/#/jobs?repo=try&revision=6785b84e807af5861e6825379072bda9fa5f763b In opt see dt4, in debug see dt6. Some noise due to browser_parsable_css also failing, but the splitconsole issue is gone with the fix.
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/50357f08249d Force computed view in splitconsole test to avoid frequent failures;r=pbro
Thank you for the quick fix: https://hg.mozilla.org/releases/mozilla-beta/rev/affeef3f9bfa52619ab64ebe4e9253a4e4486bcf Verified fixed on beta: https://treeherder.mozilla.org/#/jobs?repo=mozilla-beta&resultStatus=success%2Cusercancel%2Crunnable%2Ctestfailed%2Cbusted%2Cexception%2Cretry&group_state=expanded&searchStr=devtools&revision=affeef3f9bfa52619ab64ebe4e9253a4e4486bcf
You need to log in before you can comment on or make changes to this bug.