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

Side by Side Diff: content/test/gpu/generate_buildbot_json.py

Issue 2652223002: Disable swarming for tests where it's disabled on builder level (FYI masters) (Closed)
Patch Set: review Created 3 years, 11 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 | testing/buildbot/chromium.fyi.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 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright 2016 The Chromium Authors. All rights reserved. 2 # Copyright 2016 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """Script to generate chromium.gpu.json and chromium.gpu.fyi.json in 6 """Script to generate chromium.gpu.json and chromium.gpu.fyi.json in
7 the src/testing/buildbot directory. Maintaining these files by hand is 7 the src/testing/buildbot directory. Maintaining these files by hand is
8 too unwieldy. 8 too unwieldy.
9 """ 9 """
10 10
(...skipping 1621 matching lines...) Expand 10 before | Expand all | Expand 10 after
1632 result.pop('disabled_tester_configs') 1632 result.pop('disabled_tester_configs')
1633 if 'test' in result: 1633 if 'test' in result:
1634 result['name'] = test 1634 result['name'] = test
1635 else: 1635 else:
1636 result['test'] = test 1636 result['test'] = test
1637 if (not tester_config['swarming']) and test in NON_SWARMED_GTESTS: 1637 if (not tester_config['swarming']) and test in NON_SWARMED_GTESTS:
1638 # Need to override this result. 1638 # Need to override this result.
1639 result = copy.deepcopy(NON_SWARMED_GTESTS[test]) 1639 result = copy.deepcopy(NON_SWARMED_GTESTS[test])
1640 result['name'] = test 1640 result['name'] = test
1641 else: 1641 else:
1642 # Put the swarming dimensions in anyway. If the tester is later
1643 # swarmed, they will come in handy.
1644 if not 'swarming' in result: 1642 if not 'swarming' in result:
1645 result['swarming'] = {} 1643 result['swarming'] = {}
1646 result['swarming'].update({ 1644 result['swarming'].update({
1647 'can_use_on_swarming_builders': True, 1645 'can_use_on_swarming_builders': tester_config['swarming'],
1648 'dimension_sets': tester_config['swarming_dimensions'] 1646 'dimension_sets': tester_config['swarming_dimensions']
1649 }) 1647 })
1650 if is_android(tester_config): 1648 if is_android(tester_config):
1651 # Override the isolate target to get rid of any "_apk" suffix 1649 # Override the isolate target to get rid of any "_apk" suffix
1652 # that would be added by the recipes. 1650 # that would be added by the recipes.
1653 if 'test' in result: 1651 if 'test' in result:
1654 result['override_isolate_target'] = result['test'] 1652 result['override_isolate_target'] = result['test']
1655 else: 1653 else:
1656 result['override_isolate_target'] = result['name'] 1654 result['override_isolate_target'] = result['name']
1657 # Integrate with the unified logcat system. 1655 # Integrate with the unified logcat system.
(...skipping 86 matching lines...) Expand 10 before | Expand all | Expand 10 after
1744 test_config['asan_args'])) 1742 test_config['asan_args']))
1745 # The step name must end in 'test' or 'tests' in order for the 1743 # The step name must end in 'test' or 'tests' in order for the
1746 # results to automatically show up on the flakiness dashboard. 1744 # results to automatically show up on the flakiness dashboard.
1747 # (At least, this was true some time ago.) Continue to use this 1745 # (At least, this was true some time ago.) Continue to use this
1748 # naming convention for the time being to minimize changes. 1746 # naming convention for the time being to minimize changes.
1749 step_name = test 1747 step_name = test
1750 if not (step_name.endswith('test') or step_name.endswith('tests')): 1748 if not (step_name.endswith('test') or step_name.endswith('tests')):
1751 step_name = '%s_tests' % step_name 1749 step_name = '%s_tests' % step_name
1752 # Prepend GPU-specific flags. 1750 # Prepend GPU-specific flags.
1753 swarming = { 1751 swarming = {
1754 # Always say this is true regardless of whether the tester 1752 'can_use_on_swarming_builders': tester_config['swarming'],
1755 # supports swarming. It doesn't hurt.
1756 'can_use_on_swarming_builders': True,
1757 'dimension_sets': tester_config['swarming_dimensions'] 1753 'dimension_sets': tester_config['swarming_dimensions']
1758 } 1754 }
1759 if 'swarming' in test_config: 1755 if 'swarming' in test_config:
1760 swarming.update(test_config['swarming']) 1756 swarming.update(test_config['swarming'])
1761 result = { 1757 result = {
1762 'args': prefix_args + test_args, 1758 'args': prefix_args + test_args,
1763 'isolate_name': isolate_name, 1759 'isolate_name': isolate_name,
1764 'name': step_name, 1760 'name': step_name,
1765 'swarming': swarming, 1761 'swarming': swarming,
1766 } 1762 }
(...skipping 70 matching lines...) Expand 10 before | Expand all | Expand 10 after
1837 fp.write('\n') 1833 fp.write('\n')
1838 1834
1839 def main(): 1835 def main():
1840 generate_all_tests(FYI_WATERFALL, 'chromium.gpu.fyi.json', True, False) 1836 generate_all_tests(FYI_WATERFALL, 'chromium.gpu.fyi.json', True, False)
1841 generate_all_tests(WATERFALL, 'chromium.gpu.json', False, False) 1837 generate_all_tests(WATERFALL, 'chromium.gpu.json', False, False)
1842 generate_all_tests(V8_FYI_WATERFALL, 'client.v8.fyi.json', True, True) 1838 generate_all_tests(V8_FYI_WATERFALL, 'client.v8.fyi.json', True, True)
1843 return 0 1839 return 0
1844 1840
1845 if __name__ == "__main__": 1841 if __name__ == "__main__":
1846 sys.exit(main()) 1842 sys.exit(main())
OLDNEW
« no previous file with comments | « no previous file | testing/buildbot/chromium.fyi.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698