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

Unified Diff: scripts/slave/recipes/bisection/desktop_bisect.py

Issue 2247373002: Refactor stages 1, 2 and test_api overhaul. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 4 years, 4 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/recipes/bisection/desktop_bisect.py
diff --git a/scripts/slave/recipes/bisection/desktop_bisect.py b/scripts/slave/recipes/bisection/desktop_bisect.py
index 5aea113c87027aa1a95079ff8d050c541507f8c5..eadc91a0ff37c30899a4b38fb0ed0be5352f99a2 100644
--- a/scripts/slave/recipes/bisection/desktop_bisect.py
+++ b/scripts/slave/recipes/bisection/desktop_bisect.py
@@ -36,10 +36,13 @@ def RunSteps(api):
def GenTests(api):
- yield (api.test('basic') + api.properties.tryserver(
- path_config='kitchen',
- mastername='tryserver.chromium.perf',
- buildername='linux_perf_bisect') + api.override_step_data(
+ yield (
+ api.test('basic') +
+ api.properties.tryserver(
+ path_config='kitchen',
+ mastername='tryserver.chromium.perf',
+ buildername='linux_perf_bisect') +
+ api.override_step_data(
'git diff to analyze patch',
api.raw_io.stream_output('tools/auto_bisect/bisect.cfg')))
@@ -291,57 +294,3 @@ results-without_patch
api.step_data('buildbucket.get',
stdout=api.json.output(buildbucket_get_response)))
- bisect_config = {
- 'test_type': 'perf',
- 'command': './tools/perf/run_benchmark -v '
- '--browser=release page_cycler.intl_ar_fa_he',
- 'metric': 'warm_times/page_load_time',
- 'repeat_count': '2',
- 'max_time_minutes': '5',
- 'truncate_percent': '25',
- 'bug_id': '425582',
- 'gs_bucket': 'chrome-perf',
- 'builder_host': 'master4.golo.chromium.org',
- 'builder_port': '8341',
- }
- yield (
- api.test('basic_linux_bisect_tester_recipe') + api.properties.tryserver(
- path_config='kitchen',
- mastername='tryserver.chromium.perf',
- buildername='linux_perf_bisect') + api.step_data(
- 'saving url to temp file',
- stdout=api.raw_io.output('/tmp/dummy1')) + api.step_data(
- 'saving json to temp file',
- stdout=api.raw_io.output('/tmp/dummy2')) + api.properties(
- bisect_config=bisect_config) + api.properties(
- job_name='f7a7b4135624439cbd27fdd5133d74ec') +
- api.bisect_tester(tempfile='/tmp/dummy') + api.properties(
- parent_got_revision='1111111') + api.properties(
- parent_build_archive_url='gs://test-domain/test-archive.zip'))
-
- bisect_ret_code_config = {
- 'test_type': 'return_code',
- 'command': './tools/perf/run_benchmark -v '
- '--browser=release page_cycler.intl_ar_fa_he',
- 'metric': 'warm_times/page_load_time',
- 'repeat_count': '2',
- 'max_time_minutes': '5',
- 'truncate_percent': '25',
- 'bug_id': '425582',
- 'gs_bucket': 'chrome-perf',
- 'builder_host': 'master4.golo.chromium.org',
- 'builder_port': '8341',
- }
- yield (api.test('basic_linux_bisect_tester_recipe_ret_code') +
- api.properties.tryserver(path_config='kitchen',
- mastername='tryserver.chromium.perf',
- buildername='linux_perf_bisect') +
- api.step_data('saving url to temp file',
- stdout=api.raw_io.output('/tmp/dummy1')) + api.step_data(
- 'saving json to temp file',
- stdout=api.raw_io.output('/tmp/dummy2')) +
- api.properties(bisect_config=bisect_ret_code_config) + api.properties(
- job_name='f7a7b4135624439cbd27fdd5133d74ec') +
- api.bisect_tester(tempfile='/tmp/dummy') + api.properties(
- parent_got_revision='1111111') + api.properties(
- parent_build_archive_url='gs://test-domain/test-archive.zip'))

Powered by Google App Engine
This is Rietveld 408576698