From 98a53692fb946a8eac46e3e82257f540d1350c18 Mon Sep 17 00:00:00 2001 From: cmtice Date: Wed, 16 Apr 2014 14:48:47 -0700 Subject: Clean up Crosperf options and help. Crosperf's help was completely out of date and incorrect. Fixed that. Also, I cleaned up the options, removed some that were no longer appropriate or not ever used, and made some options global-only. Also removed all references to pyauto. We no longer use or run pyauto tests. Option changes: Completely removed outlier_range (it was not actually doing anything), key_results_only (applied only to pyauto tests), use_test_that (not needed now that we're not using pyauto) and md5sum (never used). Made the following options global-only: rm_chroot_tmp, perf_args, logging_level, show_all_results, board. Removed 'build' from global settings and made it an image-only setting (it made no sense as a global setting). I also adjusted the unittests to work with these changes. BUG=None TEST=I tested all the options in all their various positions; I tested with and wtihout caching; and I tested local and official images. I also ran the unittests. Everything works. Change-Id: I7653663f65c0a3363a1904c0220f1066e72c7f9d Reviewed-on: https://chrome-internal-review.googlesource.com/160758 Reviewed-by: Bhaskar Janakiraman Reviewed-by: Yunlian Jiang Commit-Queue: Caroline Tice Tested-by: Caroline Tice --- crosperf/translate_xbuddy.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'crosperf/translate_xbuddy.py') diff --git a/crosperf/translate_xbuddy.py b/crosperf/translate_xbuddy.py index 8a559270..92cf0230 100644 --- a/crosperf/translate_xbuddy.py +++ b/crosperf/translate_xbuddy.py @@ -22,7 +22,7 @@ def Main(xbuddy_string): os.symlink (config_path, './xbuddy_config.ini') x = xbuddy.XBuddy(manage_builds=False, static_dir='/tmp/devserver/static') build_id = x.Translate(os.path.split(xbuddy_string)) - return build_id + return build_id if __name__ == "__main__": build_id = Main(sys.argv[1]) -- cgit v1.2.3