Commit a02a2de2 authored by Haiyang Pan's avatar Haiyang Pan Committed by Commit Bot

[Perf] Replace the dimension gpu with synthetic_product_name for win-10-perf

As a followup, the dimension "os" need to be updated once crbug.com/966238 is done

Bug: 966616
Change-Id: Ic2da344c93b7f5b656a5002f514cb88a96262659
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1637603Reviewed-by: default avatarSimon Hatch <simonhatch@chromium.org>
Reviewed-by: default avatarCaleb Rouleau <crouleau@chromium.org>
Commit-Queue: Caleb Rouleau <crouleau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#665719}
parent 473608fa
...@@ -1970,9 +1970,9 @@ ...@@ -1970,9 +1970,9 @@
"can_use_on_swarming_builders": true, "can_use_on_swarming_builders": true,
"dimension_sets": [ "dimension_sets": [
{ {
"gpu": "8086:5912",
"os": "Windows-10", "os": "Windows-10",
"pool": "chrome.tests.perf" "pool": "chrome.tests.perf",
"synthetic_product_name": "OptiPlex 7050 (Dell Inc.)"
} }
], ],
"expiration": 7200, "expiration": 7200,
...@@ -2007,9 +2007,9 @@ ...@@ -2007,9 +2007,9 @@
"can_use_on_swarming_builders": true, "can_use_on_swarming_builders": true,
"dimension_sets": [ "dimension_sets": [
{ {
"gpu": "8086:5912",
"os": "Windows-10", "os": "Windows-10",
"pool": "chrome.tests.perf" "pool": "chrome.tests.perf",
"synthetic_product_name": "OptiPlex 7050 (Dell Inc.)"
} }
], ],
"expiration": 7200, "expiration": 7200,
...@@ -2044,9 +2044,9 @@ ...@@ -2044,9 +2044,9 @@
"can_use_on_swarming_builders": true, "can_use_on_swarming_builders": true,
"dimension_sets": [ "dimension_sets": [
{ {
"gpu": "8086:5912",
"os": "Windows-10", "os": "Windows-10",
"pool": "chrome.tests.perf" "pool": "chrome.tests.perf",
"synthetic_product_name": "OptiPlex 7050 (Dell Inc.)"
} }
], ],
"expiration": 7200, "expiration": 7200,
...@@ -2081,9 +2081,9 @@ ...@@ -2081,9 +2081,9 @@
"can_use_on_swarming_builders": true, "can_use_on_swarming_builders": true,
"dimension_sets": [ "dimension_sets": [
{ {
"gpu": "8086:5912",
"os": "Windows-10", "os": "Windows-10",
"pool": "chrome.tests.perf" "pool": "chrome.tests.perf",
"synthetic_product_name": "OptiPlex 7050 (Dell Inc.)"
} }
], ],
"expiration": 7200, "expiration": 7200,
...@@ -2118,9 +2118,9 @@ ...@@ -2118,9 +2118,9 @@
"can_use_on_swarming_builders": true, "can_use_on_swarming_builders": true,
"dimension_sets": [ "dimension_sets": [
{ {
"gpu": "8086:5912",
"os": "Windows-10", "os": "Windows-10",
"pool": "chrome.tests.perf" "pool": "chrome.tests.perf",
"synthetic_product_name": "OptiPlex 7050 (Dell Inc.)"
} }
], ],
"expiration": 7200, "expiration": 7200,
...@@ -2159,9 +2159,9 @@ ...@@ -2159,9 +2159,9 @@
"can_use_on_swarming_builders": true, "can_use_on_swarming_builders": true,
"dimension_sets": [ "dimension_sets": [
{ {
"gpu": "8086:5912",
"os": "Windows-10", "os": "Windows-10",
"pool": "chrome.tests.perf" "pool": "chrome.tests.perf",
"synthetic_product_name": "OptiPlex 7050 (Dell Inc.)"
} }
], ],
"expiration": 7200, "expiration": 7200,
......
...@@ -135,6 +135,11 @@ FYI_BUILDERS = { ...@@ -135,6 +135,11 @@ FYI_BUILDERS = {
# build/scripts/slave/recipe_modules/chromium_tests and must be kept in sync # build/scripts/slave/recipe_modules/chromium_tests and must be kept in sync
# to generate the correct json for each tester # to generate the correct json for each tester
# #
# The dimensions in pinpoint configs, excluding the dimension "pool",
# must be kept in sync with the dimensions here.
# This is to make sure the same type of machines are used between waterfall
# tests and pinpoint jobs
#
# On desktop builders, chromedriver is added as an additional compile target. # On desktop builders, chromedriver is added as an additional compile target.
# The perf waterfall builds this target for each commit, and the resulting # The perf waterfall builds this target for each commit, and the resulting
# ChromeDriver is archived together with Chrome for use in bisecting. # ChromeDriver is archived together with Chrome for use in bisecting.
...@@ -472,8 +477,10 @@ BUILDERS = { ...@@ -472,8 +477,10 @@ BUILDERS = {
'target_bits': 64, 'target_bits': 64,
'dimension': { 'dimension': {
'pool': 'chrome.tests.perf', 'pool': 'chrome.tests.perf',
# TODO Add more specific windows version.
# See crbug.com/966238 for more detail
'os': 'Windows-10', 'os': 'Windows-10',
'gpu': '8086:5912' 'synthetic_product_name': 'OptiPlex 7050 (Dell Inc.)'
}, },
}, },
'Win 7 Perf': { 'Win 7 Perf': {
......
...@@ -11,7 +11,8 @@ from core import bot_platforms ...@@ -11,7 +11,8 @@ from core import bot_platforms
_VALID_SWARMING_DIMENSIONS = { _VALID_SWARMING_DIMENSIONS = {
'gpu', 'device_ids', 'os', 'pool', 'perf_tests', 'perf_tests_with_args', 'gpu', 'device_ids', 'os', 'pool', 'perf_tests', 'perf_tests_with_args',
'device_os', 'device_type', 'device_os_flavor', 'id'} 'device_os', 'device_type', 'device_os_flavor', 'id',
'synthetic_product_name'}
_DEFAULT_VALID_PERF_POOLS = { _DEFAULT_VALID_PERF_POOLS = {
'chrome.tests.perf', 'chrome.tests.perf-webview', 'chrome.tests.perf', 'chrome.tests.perf-webview',
'chrome.tests.perf-fyi', 'chrome.tests.perf-webview-fyi'} 'chrome.tests.perf-fyi', 'chrome.tests.perf-webview-fyi'}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment