Chromium Code Reviews| OLD | NEW |
|---|---|
| 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 348 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 359 master_dict, | 359 master_dict, |
| 360 override_bot_type='builder_tester')) | 360 override_bot_type='builder_tester')) |
| 361 | 361 |
| 362 if enable_gpu_tests: | 362 if enable_gpu_tests: |
| 363 tests.extend(api.gpu.create_tests( | 363 tests.extend(api.gpu.create_tests( |
| 364 bot_update_step.presentation.properties['got_revision'], | 364 bot_update_step.presentation.properties['got_revision'], |
| 365 bot_update_step.presentation.properties['got_webkit_revision'], | 365 bot_update_step.presentation.properties['got_webkit_revision'], |
| 366 enable_swarming=True, | 366 enable_swarming=True, |
| 367 swarming_dimension_sets=CHROMIUM_GPU_DIMENSION_SETS[master][builder])) | 367 swarming_dimension_sets=CHROMIUM_GPU_DIMENSION_SETS[master][builder])) |
| 368 | 368 |
| 369 amp_tests = list(bot_config.get('amp_tests', [])) | |
|
navabi
2015/04/22 16:16:48
This isn't doing what I expected for the 'Android
Paweł Hajdan Jr.
2015/04/22 16:25:10
That's because bot_config here refers to the trybo
| |
| 370 | |
| 369 # TODO(phajdan.jr): Remove special case for layout tests. | 371 # TODO(phajdan.jr): Remove special case for layout tests. |
| 370 # This could be done by moving layout tests to main waterfall. | 372 # This could be done by moving layout tests to main waterfall. |
| 371 affected_files = api.tryserver.get_files_affected_by_patch() | 373 affected_files = api.tryserver.get_files_affected_by_patch() |
| 372 if (any([f.startswith('third_party/WebKit') for f in affected_files]) and | 374 if (any([f.startswith('third_party/WebKit') for f in affected_files]) and |
| 373 buildername in CHROMIUM_BLINK_TESTS_BUILDERS): | 375 buildername in CHROMIUM_BLINK_TESTS_BUILDERS): |
| 374 tests.append(api.chromium.steps.BlinkTest()) | 376 tests.append(api.chromium.steps.BlinkTest()) |
| 375 | 377 |
| 376 compile_targets, tests_including_triggered = \ | 378 compile_targets, tests_including_triggered = \ |
| 377 api.chromium_tests.get_compile_targets_and_tests( | 379 api.chromium_tests.get_compile_targets_and_tests( |
| 378 bot_config['mastername'], | 380 bot_config['mastername'], |
| 379 bot_config['buildername'], | 381 bot_config['buildername'], |
| 380 master_dict, | 382 master_dict, |
| 381 override_bot_type='builder_tester', | 383 override_bot_type='builder_tester', |
| 382 override_tests=tests) | 384 override_tests=(tests + amp_tests)) |
| 383 | 385 |
| 384 requires_compile, _, compile_targets = \ | 386 requires_compile, _, compile_targets = \ |
| 385 api.chromium_tests.analyze( | 387 api.chromium_tests.analyze( |
| 386 affected_files, | 388 affected_files, |
| 387 all_compile_targets(api, tests + tests_including_triggered), | 389 all_compile_targets(api, tests + tests_including_triggered), |
| 388 compile_targets, | 390 compile_targets, |
| 389 'trybot_analyze_config.json') | 391 'trybot_analyze_config.json') |
| 390 | 392 |
| 391 if not requires_compile: | 393 if not requires_compile: |
| 392 return | 394 return |
| 393 | 395 |
| 394 tests = tests_in_compile_targets(api, compile_targets, tests) | 396 tests = tests_in_compile_targets(api, compile_targets, tests) |
| 397 amp_tests = tests_in_compile_targets(api, compile_targets, amp_tests) | |
| 395 tests_including_triggered = tests_in_compile_targets( | 398 tests_including_triggered = tests_in_compile_targets( |
| 396 api, compile_targets, tests_including_triggered) | 399 api, compile_targets, tests_including_triggered) |
| 397 | 400 |
| 398 api.chromium_tests.compile_specific_targets( | 401 api.chromium_tests.compile_specific_targets( |
| 399 bot_config['mastername'], | 402 bot_config['mastername'], |
| 400 bot_config['buildername'], | 403 bot_config['buildername'], |
| 401 bot_update_step, | 404 bot_update_step, |
| 402 master_dict, | 405 master_dict, |
| 403 test_spec, | 406 test_spec, |
| 404 compile_targets, | 407 compile_targets, |
| 405 tests_including_triggered, | 408 tests_including_triggered, |
| 406 override_bot_type='builder_tester') | 409 override_bot_type='builder_tester') |
| 407 | 410 |
| 408 if not tests: | 411 if not tests: |
| 409 return | 412 return |
| 410 | 413 |
| 411 api.chromium_tests.run_tests_and_deapply_as_needed(mastername, api, tests, | 414 api.chromium_tests.run_tests_and_deapply_as_needed(mastername, api, tests, |
| 412 bot_update_step) | 415 bot_update_step, amp_tests) |
| 413 | 416 |
| 414 | 417 |
| 415 def GenSteps(api): | 418 def GenSteps(api): |
| 416 with api.tryserver.set_failure_hash(): | 419 with api.tryserver.set_failure_hash(): |
| 417 return _GenStepsInternal(api) | 420 return _GenStepsInternal(api) |
| 418 | 421 |
| 419 | 422 |
| 420 def _sanitize_nonalpha(text): | 423 def _sanitize_nonalpha(text): |
| 421 return ''.join(c if c.isalnum() else '_' for c in text) | 424 return ''.join(c if c.isalnum() else '_' for c in text) |
| 422 | 425 |
| (...skipping 502 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 925 ) | 928 ) |
| 926 | 929 |
| 927 yield step_failure(mastername='tryserver.chromium.linux', | 930 yield step_failure(mastername='tryserver.chromium.linux', |
| 928 buildername='android_clang_dbg_recipe', | 931 buildername='android_clang_dbg_recipe', |
| 929 steps=['compile (with patch)'], | 932 steps=['compile (with patch)'], |
| 930 tryserver=True) | 933 tryserver=True) |
| 931 yield step_failure(mastername='tryserver.chromium.linux', | 934 yield step_failure(mastername='tryserver.chromium.linux', |
| 932 buildername='android_clang_dbg_recipe', | 935 buildername='android_clang_dbg_recipe', |
| 933 steps=['compile (with patch)', 'compile (without patch)'], | 936 steps=['compile (with patch)', 'compile (without patch)'], |
| 934 tryserver=True) | 937 tryserver=True) |
| OLD | NEW |