Make --debug a remote option.
So that --pass-through='--debug' is not necessary for running debug
--buildbot runs on remote trybot.
BUG=None
TEST=Ran trybot
Change-Id: Ic1f88061e70ec874617e41ddc1f4a2d93522d2ad
Reviewed-on: https://gerrit.chromium.org/gerrit/27206
Reviewed-by: David James <davidjames@chromium.org>
Commit-Ready: Ryan Cui <rcui@chromium.org>
Tested-by: Ryan Cui <rcui@chromium.org>
diff --git a/scripts/cbuildbot.py b/scripts/cbuildbot.py
index 5a72558..52fb336 100644
--- a/scripts/cbuildbot.py
+++ b/scripts/cbuildbot.py
@@ -897,8 +897,8 @@
# Debug options
group = CustomGroup(parser, "Debug Options")
- group.add_option('--debug', action='store_true', default=None,
- help='Override some options to run as a developer.')
+ group.add_remote_option('--debug', action='store_true', default=None,
+ help='Override some options to run as a developer.')
group.add_option('--dump_config', action='store_true', dest='dump_config',
default=False,
help='Dump out build config options, and exit.')
@@ -1075,14 +1075,14 @@
print 'Submitting tryjob...'
tryjob = remote_try.RemoteTryJob(options, args, patch_pool.local_patches)
- tryjob.Submit(testjob=options.test_tryjob, dryrun=options.debug)
+ tryjob.Submit(testjob=options.test_tryjob, dryrun=False)
print 'Tryjob submitted!'
print ('Go to %s to view the status of your job.'
% tryjob.GetTrybotWaterfallLink())
if options.debug:
print
- print "Keep in mind that you had --debug enabled, thus nothing was"
- print "actually submitted."
+ print "Keep in mind that you actually submitted a tryjob run with "
+ print "--debug enabled."
sys.exit(0)
elif (not options.buildbot and not options.remote_trybot
and not options.resume and not options.local):