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

Side by Side Diff: git_cl.py

Issue 1587893003: git cl try: omit triggered bots from cq.cfg default list (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/depot_tools
Patch Set: Review Created 4 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 | Annotate | Revision Log
« no previous file with comments | « commit_queue.py ('k') | no next file » | 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 (c) 2012 The Chromium Authors. All rights reserved. 2 # Copyright (c) 2012 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 # Copyright (C) 2008 Evan Martin <martine@danga.com> 6 # Copyright (C) 2008 Evan Martin <martine@danga.com>
7 7
8 """A git-command for integrating reviews on Rietveld.""" 8 """A git-command for integrating reviews on Rietveld."""
9 9
10 from distutils.version import LooseVersion 10 from distutils.version import LooseVersion
(...skipping 3197 matching lines...) Expand 10 before | Expand all | Expand 10 after
3208 sys.stdout) 3208 sys.stdout)
3209 3209
3210 if not options.bot: 3210 if not options.bot:
3211 # Get try masters from cq.cfg if any. 3211 # Get try masters from cq.cfg if any.
3212 # TODO(tandrii): some (but very few) projects store cq.cfg in different 3212 # TODO(tandrii): some (but very few) projects store cq.cfg in different
3213 # location. 3213 # location.
3214 cq_cfg = os.path.join(change.RepositoryRoot(), 3214 cq_cfg = os.path.join(change.RepositoryRoot(),
3215 'infra', 'config', 'cq.cfg') 3215 'infra', 'config', 'cq.cfg')
3216 if os.path.exists(cq_cfg): 3216 if os.path.exists(cq_cfg):
3217 masters = {} 3217 masters = {}
3218 cq_masters = commit_queue.get_master_builder_map(cq_cfg) 3218 cq_masters = commit_queue.get_master_builder_map(
3219 cq_cfg, include_experimental=False, include_triggered=False)
3219 for master, builders in cq_masters.iteritems(): 3220 for master, builders in cq_masters.iteritems():
3220 for builder in builders: 3221 for builder in builders:
3221 # Skip presubmit builders, because these will fail without LGTM. 3222 # Skip presubmit builders, because these will fail without LGTM.
3222 if 'presubmit' not in builder.lower(): 3223 if 'presubmit' not in builder.lower():
3223 masters.setdefault(master, {})[builder] = ['defaulttests'] 3224 masters.setdefault(master, {})[builder] = ['defaulttests']
3224 if masters: 3225 if masters:
3225 return masters 3226 return masters
3226 3227
3227 if not options.bot: 3228 if not options.bot:
3228 parser.error('No default try builder to try, use --bot') 3229 parser.error('No default try builder to try, use --bot')
(...skipping 462 matching lines...) Expand 10 before | Expand all | Expand 10 after
3691 if __name__ == '__main__': 3692 if __name__ == '__main__':
3692 # These affect sys.stdout so do it outside of main() to simplify mocks in 3693 # These affect sys.stdout so do it outside of main() to simplify mocks in
3693 # unit testing. 3694 # unit testing.
3694 fix_encoding.fix_encoding() 3695 fix_encoding.fix_encoding()
3695 colorama.init() 3696 colorama.init()
3696 try: 3697 try:
3697 sys.exit(main(sys.argv[1:])) 3698 sys.exit(main(sys.argv[1:]))
3698 except KeyboardInterrupt: 3699 except KeyboardInterrupt:
3699 sys.stderr.write('interrupted\n') 3700 sys.stderr.write('interrupted\n')
3700 sys.exit(1) 3701 sys.exit(1)
OLDNEW
« no previous file with comments | « commit_queue.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698