ubuntu-emulator create needs --developer-mode and --password options

Bug #1365990 reported by Oliver Grawert
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
goget-ubuntu-touch (Ubuntu)
Fix Released
High
Sergio Schvezov

Bug Description

with the new adbd landed there is no way to make the emulator come up in developer mode on first boot, the create command should grow --developer-mode and --password options like ubuntu-device-flash has

Related branches

Oliver Grawert (ogra)
description: updated
Changed in goget-ubuntu-touch (Ubuntu):
importance: Undecided → High
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in goget-ubuntu-touch (Ubuntu):
status: New → Confirmed
Revision history for this message
Martin Pitt (pitti) wrote :

It's more than that -- I explicitly chose a PIN in the wizard, and also tried to disable/re-enable developer mode (it was on by default) in system-settings, but adb never comes up. It always appears as "offline" in adb devices. My mako is running RTM so I can't test there (as RTM didn't get the new restricted adb yet), but it can be seen in the emulator with "sudo ubuntu-emulator create --arch=i386 --channel=devel-proposed devel-proposed"

Revision history for this message
Oliver Grawert (ogra) wrote :

the offline state is most likely a udev rule issue the 70 priority we use in the adbd package seems to get overridden by something higher ...
android-tools_4.2.2+git20130218-3ubuntu30 moves the rule to prio 98, that should fix it ... (should be in the next utopic-proposed image (225))

Revision history for this message
Sergio Schvezov (sergiusens) wrote :

adb is enabled by default in the emulator and after setting a password it is still being seen as offline. Given the emulator is a specific beast and the more I think about it, a default password and --developer-mode make little sense on a "developer" tool.

I would say that we should just add logic in the "developer mode" code to just enable adb by default if the readonly device property is the emulator. Similar logic lives within powerd (for different purposes).

This would gain us the added benefit of not needing yet another hack when the "only enable developer mode after unlock" security feature lands.

Security wise, a host having access to it's emulator instance is of little "security" importance in my mind, more so, we won't need to maintain spread out hacks to support this.

fwiw, adb is enabled by default on the android emulator (build time) but not on actual products.

Revision history for this message
Oliver Grawert (ogra) wrote :

not sure it still actually is enabled ... the init.rc hacks get automatically removed on container startup and there is an upstart job now that checks for the password prior to starting android-tools-adbd (so that real devices dont end up with the gadget enabled but no daemon listening, which causes a broken state)

i wonder if we need some special treatment here ...

tags: added: rtm14
tags: added: touch-2014-09-11
Changed in goget-ubuntu-touch (Ubuntu):
assignee: nobody → Sergio Schvezov (sergiusens)
status: Confirmed → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package goget-ubuntu-touch - 0.4+14.10.20140912-0ubuntu1

---------------
goget-ubuntu-touch (0.4+14.10.20140912-0ubuntu1) utopic; urgency=low

  [ Sergio Schvezov ]
  * ubuntu-emulator: Add a default password for phablet on 'create'.
    (LP: #1365990)
 -- Ubuntu daily release <email address hidden> Fri, 12 Sep 2014 00:38:40 +0000

Changed in goget-ubuntu-touch (Ubuntu):
status: In Progress → Fix Released
Revision history for this message
Martin Pitt (pitti) wrote :

Thanks Sergio! 0000 is indeed not hard to come up with, and it's the one that autopkgtest's adb (i. e. touch) runner already tries :)

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.