aboutsummaryrefslogtreecommitdiff
path: root/crosperf/experiment_factory.py
diff options
context:
space:
mode:
authorZhizhou Yang <zhizhouy@google.com>2019-10-16 17:45:30 -0700
committerchrome-bot <chrome-bot@chromium.org>2019-10-27 05:16:16 -0700
commit45582b59e386953ecfc25a6b19ca23e18c550eef (patch)
treefe65d9944832d3c3731492f38b8e6237ac70711f /crosperf/experiment_factory.py
parente2546d5a35c3f400a74d1902992f635ade141f90 (diff)
downloadtoolchain-utils-45582b59e386953ecfc25a6b19ca23e18c550eef.tar.gz
crosperf: Add option to not download debug symbols
This patch introduces a new option --download_debug to crosperf. We will only download debug symbols when this option is set to True. This will save space when perf test users doesn't need it. TEST=passed unittests. BUG=chromium:1010294 Change-Id: Ic80958377f1c25cb04e3b7b56b40fafcada6d2d8 Reviewed-on: https://chromium-review.googlesource.com/1865958 Tested-by: Zhizhou Yang <zhizhouy@google.com> Commit-Ready: Zhizhou Yang <zhizhouy@google.com> Legacy-Commit-Queue: Commit Bot <commit-bot@chromium.org> Reviewed-by: Denis Nikitin <denik@chromium.org>
Diffstat (limited to 'crosperf/experiment_factory.py')
-rw-r--r--crosperf/experiment_factory.py6
1 files changed, 5 insertions, 1 deletions
diff --git a/crosperf/experiment_factory.py b/crosperf/experiment_factory.py
index 5b4d4b0d..1758f0c7 100644
--- a/crosperf/experiment_factory.py
+++ b/crosperf/experiment_factory.py
@@ -137,6 +137,10 @@ class ExperimentFactory(object):
remote = new_remote
rm_chroot_tmp = global_settings.GetField('rm_chroot_tmp')
perf_args = global_settings.GetField('perf_args')
+ download_debug = global_settings.GetField('download_debug')
+ # Do not download debug symbols when perf_args is not specified.
+ if not perf_args and download_debug:
+ download_debug = False
acquire_timeout = global_settings.GetField('acquire_timeout')
cache_dir = global_settings.GetField('cache_dir')
cache_only = global_settings.GetField('cache_only')
@@ -433,7 +437,7 @@ class ExperimentFactory(object):
raise RuntimeError("Can not have empty 'build' field!")
image, autotest_path, debug_path = label_settings.GetXbuddyPath(
build, autotest_path, debug_path, board, chromeos_root, log_level,
- perf_args)
+ download_debug)
cache_dir = label_settings.GetField('cache_dir')
chrome_src = label_settings.GetField('chrome_src')