Closed Bug 836973 Opened 9 years ago Closed 9 years ago

Marionette actor not getting connected to debugger

Categories

(Testing :: Marionette, defect)

defect
Not set
normal

Tracking

(blocking-b2g:tef+, firefox19 wontfix, firefox20 wontfix, firefox21 fixed, b2g18 fixed, b2g18-v1.0.0 fixed, b2g18-v1.0.1 fixed)

VERIFIED FIXED
mozilla21
blocking-b2g tef+
Tracking Status
firefox19 --- wontfix
firefox20 --- wontfix
firefox21 --- fixed
b2g18 --- fixed
b2g18-v1.0.0 --- fixed
b2g18-v1.0.1 --- fixed

People

(Reporter: jgriffin, Assigned: dhylands)

References

Details

(Keywords: dev-doc-needed)

Attachments

(1 file)

There's some problem which is causing the Marionette actor not to get connected to the debugger.  Attempting to send any Marionette message results in:

response {u'message': u'Actor "conn4.1" does not recognize the packet type "getMarionetteID"', u'from': u'root', u'error': u'unrecognizedPacketType'}

There have been no recent changes to Marionette or the debugger that would explain this.  In addition, we only see this error on tests running on unagis and pandas.
Blocks: 836745
Assignee: nobody → dhylands
I'm going to propose that if MARIONETTE is enabled, then adb will be always enabled, and disconnected from the remote-debugger setting.

The crux of the problem is that marionette wants adb on, but the remote debugger off.

What this means is that the remote debugger will always default to being off (so more or less reversing the gaia change mentioned in comment 1).

For non-user builds adb will always be on.
For user builds adb will be coupled to the remote debugger setting.
Nominating for tef? since this blocks bug 836621. Our partners run marionette tests, so they'll need/want this.
Blocks: 836621
blocking-b2g: --- → tef?
We do automated testing on -userdebug fyi.
Depends on: 837001
Blocks: 837001
No longer depends on: 837001
This forces adb to be on when marionette is enabled (marionette requires that the remote-debugger be disabled).
Attachment #708903 - Flags: review?(fabrice)
Attachment #708903 - Flags: review?(fabrice) → review+
https://hg.mozilla.org/mozilla-central/rev/c46098d2964c
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla21
Blocks another tef+ blocker.
blocking-b2g: tef? → tef+
Depends on: 836770
dev-doc-needed is to document the fact that adb is always on for non-user builds (i.e. eng and userdebug) and for user (i.e. production/dogfooding) builds adb is controlled via the remote- debugger setting.
Keywords: dev-doc-needed
Verified FIXED; our builds are back and running fine with the fix, thanks!
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.