diff options
author | Caroline Tice <cmtice@google.com> | 2018-05-24 15:44:12 -0700 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2018-05-25 20:32:12 -0700 |
commit | 372e1642e413f6d3f0047c852db95b90f76d7e6d (patch) | |
tree | 28c3da4c20db6e150fe38e93b2eb78dc438a34dc /buildbot_test_toolchains.py | |
parent | e2a920a2de9c03553db50be9484ca802d0a80dae (diff) | |
download | toolchain-utils-372e1642e413f6d3f0047c852db95b90f76d7e6d.tar.gz |
toolchain_utils: Use external chrome source tree for telemetry.
When generating the nightly performance tests, specify an external
Chrome source tree, since Chrome sync'ing is broken inside
Chrome OS. Also remove unnecessary code that references a file that
no longer exists.
BUG=chromium:846508
TEST=Successfully ran nightly tests with these changes on Chrotomation2.
Change-Id: I1cba94f82ad6ffb39c941a5dc3cb8f95cb833d20
Reviewed-on: https://chromium-review.googlesource.com/1072870
Commit-Ready: ChromeOS CL Exonerator Bot <chromiumos-cl-exonerator@appspot.gserviceaccount.com>
Tested-by: Caroline Tice <cmtice@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Diffstat (limited to 'buildbot_test_toolchains.py')
-rwxr-xr-x | buildbot_test_toolchains.py | 51 |
1 files changed, 35 insertions, 16 deletions
diff --git a/buildbot_test_toolchains.py b/buildbot_test_toolchains.py index 19032191..eb137937 100755 --- a/buildbot_test_toolchains.py +++ b/buildbot_test_toolchains.py @@ -135,16 +135,6 @@ class ToolchainComparator(object): return nonafdo_image return '' - def _FinishSetup(self): - """Make sure testing_rsa file is properly set up.""" - # Fix protections on ssh key - command = ('chmod 600 /var/cache/chromeos-cache/distfiles/target' - '/chrome-src-internal/src/third_party/chromite/ssh_keys' - '/testing_rsa') - ret_val = self._ce.ChrootRunCommand(self._chromeos_root, command) - if ret_val != 0: - raise RuntimeError('chmod for testing_rsa failed') - def _TestImages(self, trybot_image, vanilla_image, nonafdo_image): """Create crosperf experiment file. @@ -185,7 +175,17 @@ class ToolchainComparator(object): f.write(experiment_tests) # Now add vanilla to test file. - official_image = """ + if os.getlogin() == ROLE_ACCOUNT: + official_image = """ + vanilla_image { + chromeos_root: %s + build: %s + compiler: llvm + chrome_src: /usr/local/google/crostc/chrome-src-internal + } + """ % (self._chromeos_root, vanilla_image) + else: + official_image = """ vanilla_image { chromeos_root: %s build: %s @@ -196,7 +196,17 @@ class ToolchainComparator(object): # Now add non-AFDO image to test file. if nonafdo_image: - official_nonafdo_image = """ + if os.getlogin() == ROLE_ACCOUNT: + official_nonafdo_image = """ + nonafdo_image { + chromeos_root: %s + build: %s + compiler: llvm + chrome_src: /usr/local/google/crostc/chrome-src-internal + } + """ % (self._chromeos_root, nonafdo_image) + else: + official_nonafdo_image = """ nonafdo_image { chromeos_root: %s build: %s @@ -209,7 +219,19 @@ class ToolchainComparator(object): # Reuse autotest files from vanilla image for trybot images autotest_files = os.path.join('/tmp', vanilla_image, 'autotest_files') - experiment_image = """ + if os.getlogin() == ROLE_ACCOUNT: + experiment_image = """ + %s { + chromeos_root: %s + build: %s + autotest_path: %s + compiler: %s + chrome_src: /usr/local/google/crostc/chrome-src-internal + } + """ % (label_string, self._chromeos_root, trybot_image, + autotest_files, compiler_string) + else: + experiment_image = """ %s { chromeos_root: %s build: %s @@ -279,9 +301,6 @@ class ToolchainComparator(object): print('vanilla_image: %s' % vanilla_image) print('nonafdo_image: %s' % nonafdo_image) - if os.getlogin() == ROLE_ACCOUNT: - self._FinishSetup() - self._TestImages(trybot_image, vanilla_image, nonafdo_image) self._SendEmail() return 0 |