Closed Bug 1680114 Opened 4 years ago Closed 4 years ago

Set fission.autostart=false when moz:debuggerAddress is requested

Categories

(Testing :: geckodriver, task, P1)

Default
task

Tracking

(firefox85 fixed)

RESOLVED FIXED
85 Branch
Tracking Status
firefox85 --- fixed

People

(Reporter: whimboo, Assigned: whimboo)

References

(Depends on 1 open bug)

Details

Attachments

(3 files, 1 obsolete file)

Similar to bug 1680101 for Puppeteer the Remote Agent needs to force disable Fission until it is Fission compatible (bug 1600054) . We have to do that because Fission gets partially enabled for Nightly users now, and this would break the Remote Agent's functionality.

So if the moz:debuggerAddress is set to true the preference fission.autostart needs to be set to false.

Pushed by hskupin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/b3be38cb8fe6 [wdspec] Session fixture has to deep update custom capabilities. r=webdriver-reviewers,jgraham https://hg.mozilla.org/integration/autoland/rev/ac01524b0fbd [geckodriver] Set fission.autostart=false when moz:debuggerAddress is requested. r=webdriver-reviewers,jgraham
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/26741 for changes under testing/web-platform/tests
Upstream PR was closed without merging

So the problem is that for Fission jobs in CI we will always have fission.autostart set to true by --enable-fission. That means that the preference is force-fully set, and geckodriver is not going to disable Fission. That clearly breaks the tests. As such this specific test cannot be run for Fission jobs in CI.

Flags: needinfo?(hskupin)
Pushed by hskupin@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6e45b8c280e0 [wdspec] Session fixture has to deep update custom capabilities. r=webdriver-reviewers,jgraham https://hg.mozilla.org/integration/autoland/rev/c22c3f6c8ead [geckodriver] Set fission.autostart=false when moz:debuggerAddress is requested. r=webdriver-reviewers,jgraham
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 85 Branch

(In reply to Web Platform Test Sync Bot (Matrix: #interop:mozilla.org) from comment #6)

Upstream PR was closed without merging

James, shouldn't the Github PR be reopened automatically?

Flags: needinfo?(james)
Regressions: 1680960

The bot did reopen it, but it's failing presumably due to the issue in https://phabricator.services.mozilla.com/D99253 Maybe I should just attach that commit to this bug.

Flags: needinfo?(james)
Attachment #9192373 - Attachment is obsolete: true
Pushed by james@hoppipolla.co.uk: https://hg.mozilla.org/integration/autoland/rev/d3fa09337a9a Fix use of iteritems in WebDriver helpers, r=whimboo,webdriver-reviewers

(In reply to James Graham [:jgraham] from comment #11)

The bot did reopen it, but it's failing presumably due to the issue in https://phabricator.services.mozilla.com/D99253 Maybe I should just attach that commit to this bug.

Given that your follow-up landed on mozilla-central I assume some manual work is needed from your side to get both merged?

Flags: needinfo?(james)
Upstream PR was closed without merging
Flags: needinfo?(james)
Upstream PR merged by jgraham
Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6b3be0b90285 [wpt PR 26741] - [Gecko Bug 1680114] [wdspec] Session fixture has to deep update custom capabilities., a=testonly
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: