The 660/root:chronos permissions are assigned by udev and so if udev takes too long to initialize the device, they might not get assigned before your application tries to open the device node. It's weird that udev would delay on this but it could be fixed by removing that silly polling bit in DeviceJailControl::AddDevice and doing it with a udev_monitor.

Is this reproducible every time? (If you extend the sleep time in DeviceJailControl, does it go away?)

as mentioned over chat, that particular timing aspect doesn't matter as i can launch a shell and have the same problem after waiting ~forever.  the perms look fine, but `cat` never works.

running adb outside of the container works fine.

i'll check my samus.

I think I was able to repro this on my cyan, so I'll start looking for what's up there.

It doesn't seem specific to device_jail_fs. Trying to create device_jails with device_jail_utility also results in the same "udev is taking a while to initialize..." warning.

I guess I'll have to bisect to figure out when this started happening.

Did you get a chance to bisect this one?

I didn't. I'll do it today.

I'm not exactly sure how to proceed with this if it's a udev issue, though.

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/ded807e3150d4fdc16b37b674fe7f3faa0875759

commit ded807e3150d4fdc16b37b674fe7f3faa0875759
Author: Eric Caruso <ejcaruso@chromium.org>
Date: Thu Feb 01 06:38:57 2018

permission_broker: fix wrong #define

USE_DEVICE_JAIL was deprecated a long time ago. Change it to
USE_CONTAINERS which we should actually be using now.

BUG= chromium:798260 
TEST=grab usb devices from vont

Change-Id: I097932612894fae37aae3836acf5e18657236e4b
Reviewed-on: https://chromium-review.googlesource.com/896113
Commit-Ready: Eric Caruso <ejcaruso@chromium.org>
Tested-by: Eric Caruso <ejcaruso@chromium.org>
Reviewed-by: Dylan Reid <dgreid@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[modify] https://crrev.com/ded807e3150d4fdc16b37b674fe7f3faa0875759/permission_broker/permission_broker.cc

Requesting merge for the CL in comment #7 to M65 and M64.

If the feature in question fails to initialize, it does not affect anything else on the system, so it's extremely low-risk and it fixes a pretty major regression for the generic containers feature.

This bug requires manual review: Request affecting a post-stable build
Please contact the milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), kbleicher@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Your change meets the bar and is auto-approved for M65. Please go ahead and merge the CL to branch 3325 manually. Please contact milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/334bd4995057cee1a7eb204c57a500219599cb9c

commit 334bd4995057cee1a7eb204c57a500219599cb9c
Author: Eric Caruso <ejcaruso@chromium.org>
Date: Fri Feb 02 18:23:36 2018

permission_broker: fix wrong #define

USE_DEVICE_JAIL was deprecated a long time ago. Change it to
USE_CONTAINERS which we should actually be using now.

BUG= chromium:798260 
TEST=grab usb devices from vont

Change-Id: I097932612894fae37aae3836acf5e18657236e4b
Reviewed-on: https://chromium-review.googlesource.com/896113
Commit-Ready: Eric Caruso <ejcaruso@chromium.org>
Tested-by: Eric Caruso <ejcaruso@chromium.org>
Reviewed-by: Dylan Reid <dgreid@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>
(cherry picked from commit ded807e3150d4fdc16b37b674fe7f3faa0875759)
Reviewed-on: https://chromium-review.googlesource.com/899842
Reviewed-by: Eric Caruso <ejcaruso@chromium.org>
Commit-Queue: Eric Caruso <ejcaruso@chromium.org>

[modify] https://crrev.com/334bd4995057cee1a7eb204c57a500219599cb9c/permission_broker/permission_broker.cc

Can I get more context on #8?  User impact if mrege declined?  We're already on stable so need a compelling reason.

adb and fastboot won't work without this change.

This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

(Any updates on merging this to 64?)

i think this is working ... at least M66 on my pixelbook can find stuff via adb again

Per review with ejcaruso@ we expect this to be very low risk (and easy to roll back if there is).  Impacts a number of boards.  Originated in M63 but not discovered until M64.  "without the patch, adb and fastboot can't open usb devices, so they can't see any attached android device".

Thus approving for M64 merge since low risk. 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/3686f3192def5015d68e59b64799cf68dbc2fc08

commit 3686f3192def5015d68e59b64799cf68dbc2fc08
Author: Eric Caruso <ejcaruso@chromium.org>
Date: Wed Feb 14 20:00:31 2018

permission_broker: fix wrong #define

USE_DEVICE_JAIL was deprecated a long time ago. Change it to
USE_CONTAINERS which we should actually be using now.

BUG= chromium:798260 
TEST=grab usb devices from vont

Change-Id: I097932612894fae37aae3836acf5e18657236e4b
Reviewed-on: https://chromium-review.googlesource.com/896113
Commit-Ready: Eric Caruso <ejcaruso@chromium.org>
Tested-by: Eric Caruso <ejcaruso@chromium.org>
Reviewed-by: Dylan Reid <dgreid@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>
(cherry picked from commit ded807e3150d4fdc16b37b674fe7f3faa0875759)
Reviewed-on: https://chromium-review.googlesource.com/919582
Reviewed-by: Eric Caruso <ejcaruso@chromium.org>
Commit-Queue: Eric Caruso <ejcaruso@chromium.org>

[modify] https://crrev.com/3686f3192def5015d68e59b64799cf68dbc2fc08/permission_broker/permission_broker.cc