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

Unified Diff: tools/perf/core/perf_data_generator.py

Issue 2822723002: [Telemetry]Do not run power.idle_platform for reference build runs (Closed)
Patch Set: Created 3 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
« tools/perf/benchmark.csv ('K') | « tools/perf/benchmark.csv ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/perf/core/perf_data_generator.py
diff --git a/tools/perf/core/perf_data_generator.py b/tools/perf/core/perf_data_generator.py
index 927965e257f249131a913e3d7c68c97188f99c2b..b75e8f4e34ca77861d9554c24a1f4384723d97ac 100755
--- a/tools/perf/core/perf_data_generator.py
+++ b/tools/perf/core/perf_data_generator.py
@@ -624,13 +624,15 @@ def generate_telemetry_tests(
swarming_dimensions, benchmark.Name(), browser_name)
isolated_scripts.append(test)
# Now create another executable for this benchmark on the reference browser
- reference_test = generate_telemetry_test(
- swarming_dimensions, benchmark.Name(),'reference')
- isolated_scripts.append(reference_test)
- if current_shard == (num_shards - 1):
- current_shard = 0
- else:
- current_shard += 1
+ # if it is not blacklisted from running on the reference browser.
+ if benchmark.Name() not in BENCHMARK_REF_BUILD_BLACKLIST:
+ reference_test = generate_telemetry_test(
+ swarming_dimensions, benchmark.Name(),'reference')
+ isolated_scripts.append(reference_test)
+ if current_shard == (num_shards - 1):
+ current_shard = 0
+ else:
+ current_shard += 1
return isolated_scripts
@@ -668,6 +670,12 @@ LEGACY_DEVICE_AFFIINITY_ALGORITHM = [
'Win 10 High-DPI Perf',
]
+# List of benchmarks that are to never be run with reference builds.
+BENCHMARK_REF_BUILD_BLACKLIST = [
+ 'power.idle_platform',
+]
+
+
def current_benchmarks(use_whitelist):
benchmarks_dir = os.path.join(src_dir(), 'tools', 'perf', 'benchmarks')
top_level_dir = os.path.dirname(benchmarks_dir)
« tools/perf/benchmark.csv ('K') | « tools/perf/benchmark.csv ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698