Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(402)

Unified Diff: scripts/slave/telemetry.py

Issue 545803002: Update buildbots to parse new telemetry JSON format. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Switched to having telemetry output json to files. Created 6 years, 3 months 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
Index: scripts/slave/telemetry.py
diff --git a/scripts/slave/telemetry.py b/scripts/slave/telemetry.py
index b486d378517f086603caa44c7a816047ed877f9b..af0804b28b3de002f42faea7989547d797941106 100755
--- a/scripts/slave/telemetry.py
+++ b/scripts/slave/telemetry.py
@@ -77,7 +77,8 @@ def _GenerateTelemetryCommandSequence(options):
# INFO level verbosity.
'-v',
# Output results in the format the buildbot expects.
- '--output-format=buildbot',
+ '--output-format',
+ 'chartjson'
]
if profile_type:
@@ -108,6 +109,7 @@ def _GenerateTelemetryCommandSequence(options):
test_args = list(common_args)
test_args.extend(browser_info)
test_args.extend(test_specification)
+ test_args.extend(['--output', options.chart_output_filename])
test_cmd = _GetPythonTestCommand(script, target, test_args,
wrapper_args=wrapper_args, fp=fp)
commands.append(test_cmd)
@@ -141,6 +143,7 @@ def _GenerateTelemetryCommandSequence(options):
'--browser-executable=%s' % ref_build,
'--output-trace-tag=_ref'])
ref_args.extend(test_specification)
+ ref_args.extend(['--output', options.ref_output_filename])
ref_cmd = _GetPythonTestCommand(script, target, ref_args, fp=fp)
commands.append(ref_cmd)
@@ -159,6 +162,10 @@ def main(argv):
callback=chromium_utils.convert_json, type='string',
nargs=1, default={},
help='factory properties in JSON format')
+ parser.add_option('--chart-output-filename',
+ help='file to save telemetry test output')
+ parser.add_option('--ref-output-filename',
+ help='file to save reference telemetry test output')
options, _ = parser.parse_args(argv[1:])
if not options.factory_properties:

Powered by Google App Engine
This is Rietveld 408576698