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

Unified Diff: scripts/slave/recipes/android/perf.py

Issue 2192563004: [Android] Move host_info from test config to recipe step for android perf bots. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: [Android] Move host_info from test config to recipe step for android perf bots. Created 4 years, 5 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/recipes/android/perf.py
diff --git a/scripts/slave/recipes/android/perf.py b/scripts/slave/recipes/android/perf.py
index 068a07e53e027fa6428f28d7b022bf533ad80215..23f1fc52813e7aff5d8b60cbbd1a0100f7fb8508 100644
--- a/scripts/slave/recipes/android/perf.py
+++ b/scripts/slave/recipes/android/perf.py
@@ -166,6 +166,9 @@ def RunSteps(api):
for apk in required_apks:
api.chromium_android.adb_install_apk(apk)
+ api.chromium_android.host_info(
+ args=api.chromium_tests.get_common_args_for_scripts())
+
test_runner = api.chromium_tests.create_test_runner(
api, builder.get('tests', []))
@@ -299,3 +302,18 @@ def GenTests(api):
target='Release') +
api.override_step_data(
'perf_test.foo', retcode=87))
+ yield (api.test('host_info_failure') +
+ api.properties.generic(
+ path_config='kitchen',
+ repo_name='src',
+ repo_url=REPO_URL,
+ mastername='chromium.perf',
+ buildername='Android Nexus5 Perf (1)',
+ parent_buildername='parent_buildername',
+ parent_buildnumber='1729',
+ parent_revision='deadbeef',
+ revision='deadbeef',
+ slavename='slavename',
+ target='Release')
+ + api.step_data('Host_Info', retcode=87))
+

Powered by Google App Engine
This is Rietveld 408576698