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

Unified Diff: mojo/tools/get_test_list.py

Issue 1420503007: Improve the command line interface for perf dashboard upload. (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 2 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
« no previous file with comments | « mojo/devtools/common/devtoolslib/perf_dashboard.py ('k') | mojo/tools/perf_test_runner.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/tools/get_test_list.py
diff --git a/mojo/tools/get_test_list.py b/mojo/tools/get_test_list.py
index 7af7feb95310bf85c159ccf1825403dc704dd2b9..6f15c1076ae32c574856a25c5e39d003182a7c8f 100755
--- a/mojo/tools/get_test_list.py
+++ b/mojo/tools/get_test_list.py
@@ -206,7 +206,7 @@ def GetTestList(config, verbose_count=0):
# Perf tests -----------------------------------------------------------------
if target_os == Config.OS_LINUX and ShouldRunTest(Config.TEST_TYPE_PERF):
- perf_id = "linux_%s" % ("debug" if config.is_debug else "release")
+ bot_name = "linux_%s" % ("debug" if config.is_debug else "release")
test_names = ["mojo_public_system_perftests",
"mojo_public_bindings_perftests",
"mojo_system_perftests"]
@@ -214,8 +214,9 @@ def GetTestList(config, verbose_count=0):
for test_name in test_names:
command = ["python",
os.path.join("mojo", "tools", "perf_test_runner.py"),
+ "--upload",
"--server-url", _PERFORMANCE_DASHBOARD_URL,
- "--perf-id", perf_id,
+ "--bot-name", bot_name,
"--test-name", test_name,
"--perf-data-path",
os.path.join(build_dir, test_name + "_perf.log")]
« no previous file with comments | « mojo/devtools/common/devtoolslib/perf_dashboard.py ('k') | mojo/tools/perf_test_runner.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698