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

Side by Side Diff: scripts/slave/recipes/bisection/android_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 unified diff | Download patch
OLDNEW
1 # Copyright (c) 2015 The Chromium Authors. All rights reserved. 1 # Copyright (c) 2015 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 from recipe_engine.types import freeze 5 from recipe_engine.types import freeze
6 6
7 DEPS = [ 7 DEPS = [
8 'auto_bisect', 8 'auto_bisect',
9 'bisect_tester', 9 'bisect_tester',
10 'depot_tools/bot_update', 10 'depot_tools/bot_update',
(...skipping 383 matching lines...) Expand 10 before | Expand all | Expand 10 after
394 'bug_id': '425582', 394 'bug_id': '425582',
395 'gs_bucket': 'chrome-perf', 395 'gs_bucket': 'chrome-perf',
396 'builder_host': 'master4.golo.chromium.org', 396 'builder_host': 'master4.golo.chromium.org',
397 'builder_port': '8341' 397 'builder_port': '8341'
398 } 398 }
399 yield (api.test('basic_recipe_' + buildername) + 399 yield (api.test('basic_recipe_' + buildername) +
400 api.properties.tryserver( 400 api.properties.tryserver(
401 path_config='kitchen', 401 path_config='kitchen',
402 mastername='tryserver.chromium.perf', 402 mastername='tryserver.chromium.perf',
403 buildername=buildername) + 403 buildername=buildername) +
404 api.step_data(
405 'saving url to temp file',
406 stdout=api.raw_io.output('/tmp/dummy1')) +
407 api.step_data(
408 'saving json to temp file',
409 stdout=api.raw_io.output('/tmp/dummy2')) +
410 api.properties( 404 api.properties(
411 bisect_config=bisect_config) + api.properties( 405 bisect_config=bisect_config) + api.properties(
412 job_name='f7a7b4135624439cbd27fdd5133d74ec') + 406 job_name='f7a7b4135624439cbd27fdd5133d74ec') +
413 api.bisect_tester(tempfile='/tmp/dummy') + 407 api.bisect_tester(tempfile='/tmp/dummy') +
414 api.properties(parent_got_revision='1111111') + api.properties( 408 api.properties(parent_got_revision='1111111') + api.properties(
415 parent_build_archive_url='gs://test-domain/test-archive.zip')) 409 parent_build_archive_url='gs://test-domain/test-archive.zip'))
416 410
417 local_bisect_config = { 411 local_bisect_config = {
418 'test_type': 'perf', 412 'test_type': 'perf',
419 'command': './tools/perf/run_benchmark -v ' 413 'command': './tools/perf/run_benchmark -v '
(...skipping 136 matching lines...) Expand 10 before | Expand all | Expand 10 after
556 api.properties.tryserver( 550 api.properties.tryserver(
557 mastername='tryserver.chromium.perf', buildername=buildername) + 551 mastername='tryserver.chromium.perf', buildername=buildername) +
558 api.properties( 552 api.properties(
559 path_config='kitchen', 553 path_config='kitchen',
560 bisect_config=local_bisect_config, 554 bisect_config=local_bisect_config,
561 job_name='f7a7b4135624439cbd27fdd5133d74ec', 555 job_name='f7a7b4135624439cbd27fdd5133d74ec',
562 local_test=True, 556 local_test=True,
563 parent_got_revision='1111111', 557 parent_got_revision='1111111',
564 parent_build_archive_url='gs://test-domain/test-archive.zip') + 558 parent_build_archive_url='gs://test-domain/test-archive.zip') +
565 api.bisect_tester(tempfile='/tmp/dummy') + 559 api.bisect_tester(tempfile='/tmp/dummy') +
566 api.step_data(
567 'Gathering reference values.saving json to temp file',
568 stdout=api.raw_io.output('/tmp/dummy3')) +
569 api.step_data(
570 'Gathering reference values.saving json to temp file (2)',
571 stdout=api.raw_io.output('/tmp/dummy4')) +
572 api.override_step_data('device_status', 560 api.override_step_data('device_status',
573 api.json.output(two_devices)) + 561 api.json.output(two_devices)) +
574 api.override_step_data('device_status (2)', 562 api.override_step_data('device_status (2)',
575 api.json.output(two_devices)) + 563 api.json.output(two_devices)) +
576 api.step_data('Resolving reference range.crrev get commit hash for ' + 564 api.step_data('Resolving reference range.crrev get commit hash for ' +
577 ('refs/heads/master@{#%s}' % local_bisect_config['bad_revision']), 565 ('refs/heads/master@{#%s}' % local_bisect_config['bad_revision']),
578 stdout=api.json.output( 566 stdout=api.json.output(
579 {'git_sha': bad_revision_hash}))+ 567 {'git_sha': bad_revision_hash}))+
580 api.step_data('Resolving reference range.crrev get commit hash for ' + 568 api.step_data('Resolving reference range.crrev get commit hash for ' +
581 ('refs/heads/master@{#%s}' % local_bisect_config[ 569 ('refs/heads/master@{#%s}' % local_bisect_config[
(...skipping 28 matching lines...) Expand all
610 api.properties.tryserver( 598 api.properties.tryserver(
611 mastername='tryserver.chromium.perf', buildername=buildername) + 599 mastername='tryserver.chromium.perf', buildername=buildername) +
612 api.properties( 600 api.properties(
613 path_config='kitchen', 601 path_config='kitchen',
614 bisect_config=local_bisect_config, 602 bisect_config=local_bisect_config,
615 job_name='f7a7b4135624439cbd27fdd5133d74ec', 603 job_name='f7a7b4135624439cbd27fdd5133d74ec',
616 local_test=True, 604 local_test=True,
617 parent_got_revision='1111111', 605 parent_got_revision='1111111',
618 parent_build_archive_url='gs://test-domain/test-archive.zip') + 606 parent_build_archive_url='gs://test-domain/test-archive.zip') +
619 api.bisect_tester(tempfile='/tmp/dummy') + 607 api.bisect_tester(tempfile='/tmp/dummy') +
620 api.step_data(
621 'Gathering reference values.saving json to temp file',
622 stdout=api.raw_io.output('/tmp/dummy3')) +
623 api.step_data(
624 'Gathering reference values.saving json to temp file (2)',
625 stdout=api.raw_io.output('/tmp/dummy4')) +
626 api.override_step_data('device_status', 608 api.override_step_data('device_status',
627 api.json.output(working_device)) + 609 api.json.output(working_device)) +
628 api.override_step_data('device_status (2)', 610 api.override_step_data('device_status (2)',
629 api.json.output(working_device)) + 611 api.json.output(working_device)) +
630 api.step_data('Resolving reference range.crrev get commit hash for ' + 612 api.step_data('Resolving reference range.crrev get commit hash for ' +
631 ('refs/heads/master@{#%s}' % local_bisect_config['bad_revision']), 613 ('refs/heads/master@{#%s}' % local_bisect_config['bad_revision']),
632 stdout=api.json.output( 614 stdout=api.json.output(
633 {'git_sha': bad_revision_hash}))+ 615 {'git_sha': bad_revision_hash}))+
634 api.step_data('Resolving reference range.crrev get commit hash for ' + 616 api.step_data('Resolving reference range.crrev get commit hash for ' +
635 ('refs/heads/master@{#%s}' % local_bisect_config[ 617 ('refs/heads/master@{#%s}' % local_bisect_config[
(...skipping 15 matching lines...) Expand all
651 api.properties.tryserver( 633 api.properties.tryserver(
652 mastername='tryserver.chromium.perf', buildername=buildername) + 634 mastername='tryserver.chromium.perf', buildername=buildername) +
653 api.properties( 635 api.properties(
654 path_config='kitchen', 636 path_config='kitchen',
655 bisect_config=local_bisect_config, 637 bisect_config=local_bisect_config,
656 job_name='f7a7b4135624439cbd27fdd5133d74ec', 638 job_name='f7a7b4135624439cbd27fdd5133d74ec',
657 local_test=True, 639 local_test=True,
658 parent_got_revision='1111111', 640 parent_got_revision='1111111',
659 parent_build_archive_url='gs://test-domain/test-archive.zip') + 641 parent_build_archive_url='gs://test-domain/test-archive.zip') +
660 api.bisect_tester(tempfile='/tmp/dummy') + 642 api.bisect_tester(tempfile='/tmp/dummy') +
661 api.step_data(
662 'Gathering reference values.saving json to temp file',
663 stdout=api.raw_io.output('/tmp/dummy3')) +
664 api.step_data(
665 'Gathering reference values.saving json to temp file (2)',
666 stdout=api.raw_io.output('/tmp/dummy4')) +
667 api.step_data(
668 'Gathering reference values.saving json to temp file (3)',
669 stdout=api.raw_io.output('/tmp/dummy5')) +
670 api.step_data(
671 'Gathering reference values.saving json to temp file (4)',
672 stdout=api.raw_io.output('/tmp/dummy6')) +
673 api.override_step_data('device_status', 643 api.override_step_data('device_status',
674 api.json.output(two_devices)) + 644 api.json.output(two_devices)) +
675 api.override_step_data('device_status (2)', 645 api.override_step_data('device_status (2)',
676 api.json.output(two_devices)) + 646 api.json.output(two_devices)) +
677 api.step_data('Resolving reference range.crrev get commit hash for ' + 647 api.step_data('Resolving reference range.crrev get commit hash for ' +
678 ('refs/heads/master@{#%s}' % local_bisect_config['bad_revision']), 648 ('refs/heads/master@{#%s}' % local_bisect_config['bad_revision']),
679 stdout=api.json.output( 649 stdout=api.json.output(
680 {'git_sha': bad_revision_hash}))+ 650 {'git_sha': bad_revision_hash}))+
681 api.step_data('Resolving reference range.crrev get commit hash for ' + 651 api.step_data('Resolving reference range.crrev get commit hash for ' +
682 ('refs/heads/master@{#%s}' % local_bisect_config[ 652 ('refs/heads/master@{#%s}' % local_bisect_config[
683 'good_revision']), 653 'good_revision']),
684 stdout=api.json.output( 654 stdout=api.json.output(
685 {'git_sha': good_revision_hash}))+ 655 {'git_sha': good_revision_hash}))+
686 api.step_data('Expanding revision range.for revisions %s:%s' % ( 656 api.step_data('Expanding revision range.for revisions %s:%s' % (
687 good_revision_hash, bad_revision_hash), 657 good_revision_hash, bad_revision_hash),
688 stdout=api.json.output([[bad_revision_hash, 'ignored'], [ 658 stdout=api.json.output([[bad_revision_hash, 'ignored'], [
689 good_revision_hash, 'ignored']])) + 659 good_revision_hash, 'ignored']])) +
690 # Simulating disconnect by raising failure and changing the output of 660 # Simulating disconnect by raising failure and changing the output of
691 # multiple_device_status 661 # multiple_device_status
692 api.step_data('Debug Info', retcode=1) + 662 api.step_data('Debug Info', retcode=1) +
693 api.override_step_data('device_status (3)', 663 api.override_step_data('device_status (3)',
694 api.json.output(working_device)) + 664 api.json.output(working_device)) +
695 api.step_data('Expanding revision range.for revisions %s:%s (2)' % ( 665 api.step_data('Expanding revision range.for revisions %s:%s (2)' % (
696 good_revision_hash, bad_revision_hash), 666 good_revision_hash, bad_revision_hash),
697 stdout=api.json.output([[bad_revision_hash, 'ignored'], [ 667 stdout=api.json.output([[bad_revision_hash, 'ignored'], [
698 good_revision_hash, 'ignored']])) + 668 good_revision_hash, 'ignored']])) +
699 api.step_data('Post bisect results', 669 api.step_data('Post bisect results',
700 stdout=api.json.output({'status_code': 200}))) 670 stdout=api.json.output({'status_code': 200})))
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698