Add option to output device status to a JSON file.

Add option to output device status to a JSON file.

BUG=392938
R=navabi@chromium.org, tonyg@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=282728

[zty, 2014-07-11 01:20:52]

ptal

[tonyg, 2014-07-11 02:19:16]

https://codereview.chromium.org/383933003/diff/1/build/android/buildbot/bb_de...
File build/android/buildbot/bb_device_status_check.py (right):

https://codereview.chromium.org/383933003/diff/1/build/android/buildbot/bb_de...
build/android/buildbot/bb_device_status_check.py:280: help='Output JSON
information into a specified file.')
We should document the format somewhere.

https://codereview.chromium.org/383933003/diff/1/build/android/buildbot/bb_de...
build/android/buildbot/bb_device_status_check.py:368: 'online_devices': devices,
Let's add expected_devices too.

Also, prior to recipes we got some other helpful info in the build step such as
the android version and device type.

"Online devices: 1. Device types ['mako'], builds ['KOT49H']"
http://chromegw/i/chromium.perf/builders/Android%20Nexus4%20Perf/builds/983

Should we add more info to this json so that could be restored?

[zty, 2014-07-11 18:58:30]

https://codereview.chromium.org/383933003/diff/1/build/android/buildbot/bb_de...
File build/android/buildbot/bb_device_status_check.py (right):

https://codereview.chromium.org/383933003/diff/1/build/android/buildbot/bb_de...
build/android/buildbot/bb_device_status_check.py:368: 'online_devices': devices,
On 2014/07/11 02:19:15, tonyg wrote:
> Let's add expected_devices too.
> 
> Also, prior to recipes we got some other helpful info in the build step such
as
> the android version and device type.
> 
> "Online devices: 1. Device types ['mako'], builds ['KOT49H']"
> http://chromegw/i/chromium.perf/builders/Android%20Nexus4%20Perf/builds/983
> 
> Should we add more info to this json so that could be restored?

Acknowledged.

[navabi, 2014-07-11 19:08:26]

lgtm

[tonyg, 2014-07-11 20:04:51]

lgtm

[zty, 2014-07-11 20:35:40]

The CQ bit was checked by zty@chromium.org

[commit-bot: I haz the power, 2014-07-11 20:36:56]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/zty@chromium.org/383933003/20001

[navabi1, 2014-07-11 21:11:03]

lgtm

[commit-bot: I haz the power, 2014-07-11 23:20:48]

Change committed as 282728