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

Side by Side 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, 1 month 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 unified diff | 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 »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright 2014 The Chromium Authors. All rights reserved. 2 # Copyright 2014 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """Central list of tests to run (as appropriate for a given config). Add tests 6 """Central list of tests to run (as appropriate for a given config). Add tests
7 to run by modifying this file. 7 to run by modifying this file.
8 8
9 Note that this file is both imported (by mojob.py) and run directly (via a 9 Note that this file is both imported (by mojob.py) and run directly (via a
10 recipe).""" 10 recipe)."""
(...skipping 188 matching lines...) Expand 10 before | Expand all | Expand 10 after
199 "--dart-pkg-dir=" + os.path.join(build_dir, "gen", "dart-pkg"), 199 "--dart-pkg-dir=" + os.path.join(build_dir, "gen", "dart-pkg"),
200 "--dart-sdk=" + os.path.join("third_party", "dart-sdk", "dart-sdk"), 200 "--dart-sdk=" + os.path.join("third_party", "dart-sdk", "dart-sdk"),
201 "--package-root=" + os.path.join(build_dir, 201 "--package-root=" + os.path.join(build_dir,
202 "gen", 202 "gen",
203 "dart-pkg", 203 "dart-pkg",
204 "packages")]) 204 "packages")])
205 205
206 # Perf tests ----------------------------------------------------------------- 206 # Perf tests -----------------------------------------------------------------
207 207
208 if target_os == Config.OS_LINUX and ShouldRunTest(Config.TEST_TYPE_PERF): 208 if target_os == Config.OS_LINUX and ShouldRunTest(Config.TEST_TYPE_PERF):
209 perf_id = "linux_%s" % ("debug" if config.is_debug else "release") 209 bot_name = "linux_%s" % ("debug" if config.is_debug else "release")
210 test_names = ["mojo_public_system_perftests", 210 test_names = ["mojo_public_system_perftests",
211 "mojo_public_bindings_perftests", 211 "mojo_public_bindings_perftests",
212 "mojo_system_perftests"] 212 "mojo_system_perftests"]
213 213
214 for test_name in test_names: 214 for test_name in test_names:
215 command = ["python", 215 command = ["python",
216 os.path.join("mojo", "tools", "perf_test_runner.py"), 216 os.path.join("mojo", "tools", "perf_test_runner.py"),
217 "--upload",
217 "--server-url", _PERFORMANCE_DASHBOARD_URL, 218 "--server-url", _PERFORMANCE_DASHBOARD_URL,
218 "--perf-id", perf_id, 219 "--bot-name", bot_name,
219 "--test-name", test_name, 220 "--test-name", test_name,
220 "--perf-data-path", 221 "--perf-data-path",
221 os.path.join(build_dir, test_name + "_perf.log")] 222 os.path.join(build_dir, test_name + "_perf.log")]
222 if config.values.get("builder_name"): 223 if config.values.get("builder_name"):
223 command += ["--builder-name", config.values["builder_name"]] 224 command += ["--builder-name", config.values["builder_name"]]
224 if config.values.get("build_number"): 225 if config.values.get("build_number"):
225 command += ["--build-number", config.values["build_number"]] 226 command += ["--build-number", config.values["build_number"]]
226 if config.values.get("master_name"): 227 if config.values.get("master_name"):
227 command += ["--master-name", config.values["master_name"]] 228 command += ["--master-name", config.values["master_name"]]
228 command += [os.path.join(build_dir, test_name)] 229 command += [os.path.join(build_dir, test_name)]
(...skipping 29 matching lines...) Expand all
258 config = Config(**json.load(args.config_file)) 259 config = Config(**json.load(args.config_file))
259 test_list = GetTestList(config) 260 test_list = GetTestList(config)
260 json.dump(test_list, args.test_list_file, indent=2) 261 json.dump(test_list, args.test_list_file, indent=2)
261 args.test_list_file.write("\n") 262 args.test_list_file.write("\n")
262 263
263 return 0 264 return 0
264 265
265 266
266 if __name__ == "__main__": 267 if __name__ == "__main__":
267 sys.exit(main()) 268 sys.exit(main())
OLDNEW
« 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