Don't report android device port paths.

Don't report android device port paths.

BUG=608427
Committed: https://chromium.googlesource.com/infra/infra/+/316fd863534efaa772b0d333cd7e57dea181506a

[bpastene, 2016-06-01 17:50:12]

bpastene@chromium.org changed reviewers:
+ stip@chromium.org

[ghost stip (do not use), 2016-06-01 18:03:37]

https://codereview.chromium.org/2023003004/diff/1/infra/services/sysmon/andro...
File infra/services/sysmon/android_device_metrics.py (right):

https://codereview.chromium.org/2023003004/diff/1/infra/services/sysmon/andro...
infra/services/sysmon/android_device_metrics.py:58: logging.warning('Found port
path %s as device id. Skipping.',
interesting. when would this come up instead of device? (I guess that might be a
swarming thing?) if we drop it, are we losing data?

[bpastene, 2016-06-01 21:43:30]

https://codereview.chromium.org/2023003004/diff/1/infra/services/sysmon/andro...
File infra/services/sysmon/android_device_metrics.py (right):

https://codereview.chromium.org/2023003004/diff/1/infra/services/sysmon/andro...
infra/services/sysmon/android_device_metrics.py:58: logging.warning('Found port
path %s as device id. Skipping.',
On 2016/06/01 18:03:37, stip wrote:
> interesting. when would this come up instead of device?

I imagine it happens when there's a protocol/usb failure with the device. It
happens a bunch to this guy (look at all the USBErrorIO errors):
https://chromium-swarm.appspot.com/restricted/bot/build342-m4

> if we drop it, are we losing data?

When it does happen, the only data we have is 'status: unavailable', which we're
unable to even map to a specific device id. So no, we are not.

[ghost stip (do not use), 2016-06-01 23:27:59]

cool. lgtm

[ghost stip (do not use), 2016-06-01 23:28:26]

we should look into tracking the 'unavailable' number as a metric though

[bpastene, 2016-06-02 00:33:58]

The CQ bit was checked by bpastene@chromium.org

[commit-bot: I haz the power, 2016-06-02 00:34:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2023003004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2023003004/1

[commit-bot: I haz the power, 2016-06-02 00:42:07]

Description was changed from

==========
Don't report android device port paths.

BUG=608427
==========

to

==========
Don't report android device port paths.

BUG=608427

Committed:
https://chromium.googlesource.com/infra/infra/+/316fd863534efaa772b0d333cd7e5...
==========

[commit-bot: I haz the power, 2016-06-02 00:42:08]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/infra/infra/+/316fd863534efaa772b0d333cd7e5...