Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1132)

Unified Diff: scripts/slave/recipe_modules/chromium_tests/chromium_perf.py

Issue 2436863004: Add new chromium recipe builder to chromium.perf (Closed)
Patch Set: Remove old file. Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: scripts/slave/recipe_modules/chromium_tests/chromium_perf.py
diff --git a/scripts/slave/recipe_modules/chromium_tests/chromium_perf.py b/scripts/slave/recipe_modules/chromium_tests/chromium_perf.py
index c8207d4bfa4de0a5a4668089e7895ebf756d99b8..9312fe2740b3ec5a966c28b09ad622273b8c630a 100644
--- a/scripts/slave/recipe_modules/chromium_tests/chromium_perf.py
+++ b/scripts/slave/recipe_modules/chromium_tests/chromium_perf.py
@@ -112,8 +112,13 @@ def _AddBuildSpec(
SPEC['builders'][name] = BuildSpec(
'chromium_perf', perf_id, platform, target_bits, enable_swarming)
- assert target_bits not in builders[platform]
- builders[platform][target_bits] = name
+
+ # TODO(martiniss): re-enable assertion once android has switched to the
+ # chromium recipe
+ # assert target_bits not in builders[platform]
+
+ if not builders[platform].get(target_bits, None):
+ builders[platform][target_bits] = name
if add_to_bisect:
SPEC['settings']['bisect_builders'].append(name)
@@ -130,6 +135,7 @@ def _AddTestSpec(name, perf_id, platform, target_bits=64,
_AddBuildSpec('Android Builder', 'android', target_bits=32)
+_AddBuildSpec('Android Temporary Compile', 'android', target_bits=32)
_AddBuildSpec('Android arm64 Builder', 'android')
_AddBuildSpec('Win Builder', 'win', target_bits=32)
_AddBuildSpec( \

Powered by Google App Engine
This is Rietveld 408576698