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

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

Issue 1104533002: Add recipe for split AMP/local CQ. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Created 5 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 unified diff | Download patch | Annotate | Revision Log
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 348 matching lines...) Expand 10 before | Expand all | Expand 10 after
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
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)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698