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

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

Issue 1920863007: Add sizes test to chromium.perf builders. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Remove unused bots and bot code. Created 4 years, 8 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/tryserver_chromium_perf.py
diff --git a/scripts/slave/recipe_modules/chromium_tests/tryserver_chromium_perf.py b/scripts/slave/recipe_modules/chromium_tests/tryserver_chromium_perf.py
index aa2ca8e9dfc3b8ed809079d1e8d10317a757e85d..8b4f4d3643fe04cb76d2045731e865d7435ff294 100644
--- a/scripts/slave/recipe_modules/chromium_tests/tryserver_chromium_perf.py
+++ b/scripts/slave/recipe_modules/chromium_tests/tryserver_chromium_perf.py
@@ -57,58 +57,10 @@ SPEC = {
chromium_perf.SPEC['builders']['Mac Builder'],
'mac_fyi_perf_bisect':
chromium_perf.SPEC['builders']['Mac Builder'],
- 'linux_perf_tester':{
- 'chromium_config': 'chromium_official',
- 'gclient_config': 'chromium',
- 'gclient_apply_config': ['chrome_internal'],
- 'parent_buildername': 'Linux Builder',
- 'chromium_config_kwargs': {
- 'BUILD_CONFIG': 'Release',
- 'TARGET_BITS': 64,
- },
- 'bot_type': 'tester',
- 'compile_targets': [
- 'chromium_builder_perf',
- ],
- 'testing': {
- 'platform': 'linux',
- },
- 'tests':[steps.BisectTest()],
- 'chromium_apply_config': ['chromium_perf']
- },
- 'linux_perf_bisector':{
- 'chromium_config': 'chromium_official',
- 'gclient_config': 'chromium',
- 'gclient_apply_config': ['chrome_internal'],
- 'parent_buildername': 'Linux Builder',
- 'chromium_config_kwargs': {
- 'BUILD_CONFIG': 'Release',
- 'TARGET_BITS': 64,
- },
- 'bot_type': 'tester',
- 'compile_targets': [
- 'chromium_builder_perf',
- ],
- 'testing': {
- 'platform': 'linux',
- },
- 'tests':[steps.BisectTest()],
- 'chromium_apply_config': ['chromium_perf']
- },
- 'win64_nv_tester':{
- 'bot_type': 'tester',
- 'chromium_config_kwargs': {
- 'BUILD_CONFIG': 'Release',
- 'TARGET_BITS': 64,
- },
- 'parent_buildername': 'Win x64 Builder',
- 'chromium_config': 'chromium_official',
- 'gclient_config': 'perf',
- 'testing': {
- 'platform': 'win',
- },
- 'tests':[steps.BisectTest()],
- },
hans 2016/04/27 15:49:42 Is this the dead code you're referring to in the d
}
}
+
+for builder_config in SPEC['builders'].itervalues():
+ if 'tests' in builder_config:
+ del builder_config['tests']

Powered by Google App Engine
This is Rietveld 408576698