commit 54da0700f1a489a365ce28077000695d13e50bec
author Ryan Cui <rcui@chromium.org> Thu Apr 19 18:38:08 2012 -0700
committer Gerrit <chrome-bot@google.com> Mon Apr 23 11:32:24 2012 -0700
tree 844a99cec63858f45477b772a54a04dd22842299
parent 55e012fe17b503fd689a0425839ee10877c16180

Show warning if in local trybot mode.

Allow user to suppress warning using --local.  The next step is to force
users to specify either --local or --remote.

BUG=chromium-os:29814
TEST=local trybot and remote trybot

Change-Id: Ie79a7a1041b845d4887b7bfee27aeac14d36b9fa
Reviewed-on: https://gerrit.chromium.org/gerrit/20648
Commit-Ready: Ryan Cui <rcui@chromium.org>
Reviewed-by: Ryan Cui <rcui@chromium.org>
Tested-by: Ryan Cui <rcui@chromium.org>

scripts/cbuildbot.py

diff --git a/scripts/cbuildbot.py b/scripts/cbuildbot.py
index 87aed98..98508ab 100644
--- a/scripts/cbuildbot.py
+++ b/scripts/cbuildbot.py
@@ -17,6 +17,7 @@
 import os
 import pprint
 import sys
+import time
 
 from chromite.buildbot import builderstage as bs
 from chromite.buildbot import cbuildbot_background as background
@@ -680,6 +681,8 @@
                     default=False,
                     help=('List the suggested trybot configs to use.  Use '
                           '--all to list all of the available configs.'))
+  parser.add_option('--local', default=False, action='store_true',
+                    help=('Specifies that this tryjob should be run locally.'))
   parser.add_option('-p', '--local-patches', action='append', default=[],
                     metavar="'<project1>[:<branch1>]...<projectN>[:<branchN>]'",
                     help=('Space-separated list of project branches with '
@@ -822,6 +825,9 @@
       cros_lib.Die('Chrome rev must not be %s if chrome_version is not set.' %
                    constants.CHROME_REV_SPEC)
 
+  if options.local and options.remote:
+   cros_lib.Die('Cannot specify both --remote and --local')
+
   if options.remote and not (options.gerrit_patches or options.local_patches):
     cros_lib.Die('Must provide patches when running with --remote.')
 
@@ -831,6 +837,9 @@
   if options.buildbot and options.remote_trybot:
     cros_lib.Die('--buildbot and --remote-trybot cannot be used together.')
 
+  if options.buildbot and (options.remote or options.local):
+   cros_lib.Die('--remote and --local do not apply when using --buildbot.')
+
   # Record whether --debug was set explicitly vs. it was inferred.
   options.debug_forced = False
   if options.debug:
@@ -882,6 +891,10 @@
   # Strip out null arguments.
   # TODO(rcui): Remove when buildbot is fixed
   args = [arg for arg in args if arg]
+  if options.list:
+    _PrintValidConfigs(options.print_all)
+    sys.exit(0)
+
   _FinishParsing(options, args)
   return options, args
 
@@ -896,10 +909,6 @@
   parser = _CreateParser()
   (options, args) = _ParseCommandLine(parser, argv)
 
-  if options.list:
-    _PrintValidConfigs(options.print_all)
-    sys.exit(0)
-
   _PostParseCheck(options, args)
 
   if options.remote:
@@ -920,6 +929,13 @@
     print ('Go to %s to view the status of your job.'
            % tryjob.GetTrybotWaterfallLink())
     sys.exit(0)
+  elif (not options.buildbot and not options.remote_trybot
+        and not options.resume and not options.local):
+    cros_lib.Warning('Running in LOCAL TRYBOT mode!  Use --remote to submit '
+                     'REMOTE tryjobs.  Use --local to suppress this message.')
+    cros_lib.Warning('Starting April 30th, --local will be required to run the '
+                     'local trybot.')
+    time.sleep(5)
 
   if args:
     # Only expecting one config