scripts/slave/recipes/cronet.expected/android_cronet_builder.json - Issue 815213002: Cause chromium_android recipe to consume device_status_check's json.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: scripts/slave/recipes/cronet.expected/android_cronet_builder.json

Issue 815213002: Cause chromium_android recipe to consume device_status_check's json. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master

Patch Set: Add missing files Created 6 years ago

Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.

Jump to:

View side-by-side diff with in-line comments

Download patch

« no previous file with comments | « scripts/slave/recipes/cronet.expected/android_cronet_armv6_builder.json ('k') | scripts/slave/recipes/cronet.expected/android_cronet_builder__dbg_.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

Index: scripts/slave/recipes/cronet.expected/android_cronet_builder.json

diff --git a/scripts/slave/recipes/cronet.expected/android_cronet_builder.json b/scripts/slave/recipes/cronet.expected/android_cronet_builder.json

index 11fc7ef52f8152ec76609da374157a7bdedf312e..ea22c5d3c616421cdf89a17265b0fcc9696cf712 100644

--- a/scripts/slave/recipes/cronet.expected/android_cronet_builder.json

+++ b/scripts/slave/recipes/cronet.expected/android_cronet_builder.json

@@ -226,12 +226,40 @@

{

"cmd": [

- "[SLAVE_BUILD]/src/build/android/buildbot/bb_device_status_check.py"

+ "[SLAVE_BUILD]/src/build/android/buildbot/bb_device_status_check.py",

+ "--json-output",

+ "/path/to/tmp/json"

"env": {

"PATH": "[SLAVE_BUILD]/src/third_party/android_tools/sdk/platform-tools:[SLAVE_BUILD]/src/build/android:%(PATH)s"

- "name": "device_status_check"

+ "name": "device_status_check",

+ "~followup_annotations": [

+ "@@@STEP_LOG_LINE@json.output@[@@@",

+ "@@@STEP_LOG_LINE@json.output@ {@@@",

+ "@@@STEP_LOG_LINE@json.output@ \"battery\": {@@@",

+ "@@@STEP_LOG_LINE@json.output@ \"AC powered\": \"false\", @@@",

+ "@@@STEP_LOG_LINE@json.output@ \"USB powered\": \"true\", @@@",

+ "@@@STEP_LOG_LINE@json.output@ \"Wireless powered\": \"false\", @@@",

+ "@@@STEP_LOG_LINE@json.output@ \"health\": \"2\", @@@",

+ "@@@STEP_LOG_LINE@json.output@ \"level\": \"100\", @@@",

+ "@@@STEP_LOG_LINE@json.output@ \"present\": \"true\", @@@",

+ "@@@STEP_LOG_LINE@json.output@ \"scale\": \"100\", @@@",

+ "@@@STEP_LOG_LINE@json.output@ \"status\": \"5\", @@@",

+ "@@@STEP_LOG_LINE@json.output@ \"technology\": \"Li-ion\", @@@",

+ "@@@STEP_LOG_LINE@json.output@ \"temperature\": \"249\", @@@",

+ "@@@STEP_LOG_LINE@json.output@ \"voltage\": \"4286\"@@@",

+ "@@@STEP_LOG_LINE@json.output@ }, @@@",

+ "@@@STEP_LOG_LINE@json.output@ \"build\": \"LRX21O\", @@@",

+ "@@@STEP_LOG_LINE@json.output@ \"build_detail\": \"google/razor/flo:5.0/LRX21O/1570415:userdebug/dev-keys\", @@@",

+ "@@@STEP_LOG_LINE@json.output@ \"imei_slice\": \"Unknown\", @@@",

+ "@@@STEP_LOG_LINE@json.output@ \"serial\": \"07a00ca4\", @@@",

+ "@@@STEP_LOG_LINE@json.output@ \"type\": \"flo\", @@@",

+ "@@@STEP_LOG_LINE@json.output@ \"wifi_ip\": \"\"@@@",

+ "@@@STEP_LOG_LINE@json.output@ }@@@",

+ "@@@STEP_LOG_LINE@json.output@]@@@",

+ "@@@STEP_LOG_END@json.output@@@"

+ ]

{

"cmd": [