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 | |
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>
-rwxr-xr-x | buildbot_test_llvm.py | 10 | ||||
-rwxr-xr-x | buildbot_test_toolchains.py | 51 |
2 files changed, 35 insertions, 26 deletions
diff --git a/buildbot_test_llvm.py b/buildbot_test_llvm.py index 5b9e6944..111068c6 100755 --- a/buildbot_test_llvm.py +++ b/buildbot_test_llvm.py @@ -109,16 +109,6 @@ class ToolchainVerifier(object): self._weekday = weekday self._reports = os.path.join(VALIDATION_RESULT_DIR, compiler, board) - 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 DoAll(self): """Main function inside ToolchainComparator class. 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 |