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

Side by Side Diff: scripts/slave/recipes/chromium_trybot.py

Issue 1176553003: chromium_trybot: use matching_exes from analyze rather than compile_targets (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 5 years, 6 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
« no previous file with comments | « no previous file | scripts/slave/recipes/chromium_trybot.expected/compile_because_of_analyze_with_filtered_compile_targets.json » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 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 infra.libs.infra_types import freeze 5 from infra.libs.infra_types import freeze
6 6
7 DEPS = [ 7 DEPS = [
8 'bot_update', 8 'bot_update',
9 'chromium', 9 'chromium',
10 'chromium_android', 10 'chromium_android',
(...skipping 403 matching lines...) Expand 10 before | Expand all | Expand 10 after
414 tests.append(api.chromium.steps.BlinkTest()) 414 tests.append(api.chromium.steps.BlinkTest())
415 415
416 compile_targets, tests_including_triggered = \ 416 compile_targets, tests_including_triggered = \
417 api.chromium_tests.get_compile_targets_and_tests( 417 api.chromium_tests.get_compile_targets_and_tests(
418 bot_config['mastername'], 418 bot_config['mastername'],
419 bot_config['buildername'], 419 bot_config['buildername'],
420 master_dict, 420 master_dict,
421 override_bot_type='builder_tester', 421 override_bot_type='builder_tester',
422 override_tests=tests) 422 override_tests=tests)
423 423
424 requires_compile, _, compile_targets = \ 424 requires_compile, matching_exes, compile_targets = \
425 api.chromium_tests.analyze( 425 api.chromium_tests.analyze(
426 affected_files, 426 affected_files,
427 all_compile_targets(api, tests + tests_including_triggered), 427 all_compile_targets(api, tests + tests_including_triggered),
428 compile_targets, 428 compile_targets,
429 'trybot_analyze_config.json') 429 'trybot_analyze_config.json')
430 430
431 if not requires_compile: 431 if not requires_compile:
432 return 432 return
433 433
434 tests = tests_in_compile_targets(api, compile_targets, tests) 434 tests = tests_in_compile_targets(api, matching_exes, tests)
435 tests_including_triggered = tests_in_compile_targets( 435 tests_including_triggered = tests_in_compile_targets(
436 api, compile_targets, tests_including_triggered) 436 api, matching_exes, tests_including_triggered)
437 437
438 api.chromium_tests.compile_specific_targets( 438 api.chromium_tests.compile_specific_targets(
439 bot_config['mastername'], 439 bot_config['mastername'],
440 bot_config['buildername'], 440 bot_config['buildername'],
441 bot_update_step, 441 bot_update_step,
442 master_dict, 442 master_dict,
443 test_spec, 443 test_spec,
444 compile_targets, 444 compile_targets,
445 tests_including_triggered, 445 tests_including_triggered,
446 override_bot_type='builder_tester') 446 override_bot_type='builder_tester')
(...skipping 542 matching lines...) Expand 10 before | Expand all | Expand 10 after
989 989
990 yield ( 990 yield (
991 api.test('swarming_paths') + 991 api.test('swarming_paths') +
992 api.properties.tryserver( 992 api.properties.tryserver(
993 mastername='tryserver.chromium.linux', 993 mastername='tryserver.chromium.linux',
994 buildername='linux_chromium_rel_ng', 994 buildername='linux_chromium_rel_ng',
995 path_config='swarming', 995 path_config='swarming',
996 ) + 996 ) +
997 api.platform.name('linux') 997 api.platform.name('linux')
998 ) 998 )
OLDNEW
« no previous file with comments | « no previous file | scripts/slave/recipes/chromium_trybot.expected/compile_because_of_analyze_with_filtered_compile_targets.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698