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>
diff --git a/buildbot_test_llvm.py b/buildbot_test_llvm.py
index 5b9e694..111068c 100755
--- a/buildbot_test_llvm.py
+++ b/buildbot_test_llvm.py
@@ -109,16 +109,6 @@
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 1903219..eb13793 100755
--- a/buildbot_test_toolchains.py
+++ b/buildbot_test_toolchains.py
@@ -135,16 +135,6 @@
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 @@
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 @@
# 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 @@
# 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 @@
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