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

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

Issue 1974703002: Re-enable sizes test on tryserver.chromium.perf. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 4 years, 7 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
« no previous file with comments | « no previous file | scripts/slave/recipes/chromium.expected/full_tryserver_chromium_perf_linux_perf_bisect_builder.json » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 724895b078072f1640d9955cef9ee41930a2f0ba..303d7f7a9e8ba9c3998db69e6c0d293065351d3f 100644
--- a/scripts/slave/recipe_modules/chromium_tests/tryserver_chromium_perf.py
+++ b/scripts/slave/recipe_modules/chromium_tests/tryserver_chromium_perf.py
@@ -11,23 +11,35 @@ from . import steps
def _Builders():
# These should build with the same configs as the main perf builders.
builders = {}
- for bisect_bot_name, perf_bot_name in _PERF_BOT_MAPPINGS.iteritems():
+
+ for bisect_bot_name, perf_bot_name in _BUILDER_MAPPINGS.iteritems():
+ config = copy.deepcopy(chromium_perf.SPEC['builders'][perf_bot_name])
+ config['tests'] = [steps.SizesStep(None, None)]
+ builders[bisect_bot_name] = config
+
+ for bisect_bot_name, perf_bot_name in _TESTER_MAPPINGS.iteritems():
config = copy.deepcopy(chromium_perf.SPEC['builders'][perf_bot_name])
del config['tests'] # Don't run the same tests as the perf waterfall.
builders[bisect_bot_name] = config
+
return builders
-_PERF_BOT_MAPPINGS = {
+_BUILDER_MAPPINGS = {
'linux_perf_bisect_builder': 'Linux Builder',
+ 'win_perf_bisect_builder': 'Win Builder',
+ 'winx64_bisect_builder': 'Win x64 Builder',
+ 'mac_perf_bisect_builder': 'Mac Builder',
+}
+
+
+_TESTER_MAPPINGS = {
'linux_perf_bisect': 'Linux Builder',
'linux_perf_cq': 'Linux Builder',
'linux_fyi_perf_bisect': 'Linux Builder',
- 'win_perf_bisect_builder': 'Win Builder',
'win_perf_bisect': 'Win Builder',
'win_8_perf_bisect': 'Win Builder',
'winx64_10_perf_bisect': 'Win x64 Builder',
- 'winx64_bisect_builder': 'Win x64 Builder',
'win_x64_perf_bisect': 'Win x64 Builder',
'winx64ati_perf_bisect': 'Win x64 Builder',
'winx64nvidia_perf_bisect': 'Win x64 Builder',
@@ -35,7 +47,6 @@ _PERF_BOT_MAPPINGS = {
'winx64_zen_perf_bisect': 'Win x64 Builder',
'winx64_10_perf_cq': 'Win x64 Builder',
'win_fyi_perf_bisect': 'Win Builder',
- 'mac_perf_bisect_builder': 'Mac Builder',
'mac_10_11_perf_bisect': 'Mac Builder',
'mac_10_10_perf_bisect': 'Mac Builder',
'mac_retina_perf_bisect': 'Mac Builder',
« no previous file with comments | « no previous file | scripts/slave/recipes/chromium.expected/full_tryserver_chromium_perf_linux_perf_bisect_builder.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698