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

Unified Diff: git_cl.py

Issue 2401943002: git cl try: remove --use-rietveld and related code. (Closed)
Patch Set: rebase Created 4 years, 2 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: git_cl.py
diff --git a/git_cl.py b/git_cl.py
index 857bf37ddd577a23991420203e08d7cd007d4eb5..0b8ac7b7c6367faee174fff9d3bb9f40d76b1d77 100755
--- a/git_cl.py
+++ b/git_cl.py
@@ -4670,10 +4670,6 @@ def CMDtry(parser, args):
# TODO(tandrii): if this even used?
group.add_option(
'-n', '--name', help='Try job name; default to current branch name')
- # TODO(tandrii): get rid of this.
- group.add_option(
- '--use-rietveld', action='store_true', default=False,
- help='DEPRECATED, NOT SUPPORTED.')
group.add_option(
'--buildbucket-host', default='cr-buildbucket.appspot.com',
help='Host of buildbucket. The default host is %default.')
@@ -4682,9 +4678,6 @@ def CMDtry(parser, args):
options, args = parser.parse_args(args)
auth_config = auth.extract_auth_config_from_options(options)
- if options.use_rietveld:
- parser.error('--use-rietveld is not longer supported.')
-
# Make sure that all properties are prop=value pairs.
bad_params = [x for x in options.properties if '=' not in x]
if bad_params:
@@ -4812,35 +4805,16 @@ def CMDtry(parser, args):
'\nWARNING Mismatch between local config and server. Did a previous '
'upload fail?\ngit-cl try always uses latest patchset from rietveld. '
'Continuing using\npatchset %s.\n' % patchset)
- if not options.use_rietveld:
- try:
- trigger_try_jobs(auth_config, cl, options, masters, 'git_cl_try')
- except BuildbucketResponseException as ex:
- print('ERROR: %s' % ex)
- return 1
- except Exception as e:
- stacktrace = (''.join(traceback.format_stack()) + traceback.format_exc())
- print('ERROR: Exception when trying to trigger try jobs: %s\n%s' %
- (e, stacktrace))
- return 1
- else:
- try:
- cl.RpcServer().trigger_distributed_try_jobs(
- cl.GetIssue(), patchset, options.name, options.clobber,
- options.revision, masters)
- except urllib2.HTTPError as e:
- if e.code == 404:
- print('404 from rietveld; '
- 'did you mean to use "git try" instead of "git cl try"?')
- return 1
- print('Tried jobs on:')
-
- for (master, builders) in sorted(masters.iteritems()):
- if master:
- print('Master: %s' % master)
- length = max(len(builder) for builder in builders)
- for builder in sorted(builders):
- print(' %*s: %s' % (length, builder, ','.join(builders[builder])))
+ try:
+ trigger_try_jobs(auth_config, cl, options, masters, 'git_cl_try')
+ except BuildbucketResponseException as ex:
+ print('ERROR: %s' % ex)
+ return 1
+ except Exception as e:
+ stacktrace = (''.join(traceback.format_stack()) + traceback.format_exc())
+ print('ERROR: Exception when trying to trigger try jobs: %s\n%s' %
+ (e, stacktrace))
+ return 1
return 0
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698