diff options
author | 2013-07-12 18:11:04 +0000 | |
---|---|---|
committer | 2013-07-12 18:11:04 +0000 | |
commit | 6fb77569596ad3e9419e9450db959949cc37576a (patch) | |
tree | bb3038047b3e14bc56dd793f473fb511f769132f /tools | |
parent | cfe8aa6f560e0eb1d337d6d19432138aa9be9fd7 (diff) |
If a user requests a try on "all" builders, ask for confirmation
Addresses https://code.google.com/p/skia/issues/detail?id=1208
(SkipBuildbotRuns)
R=epoger@google.com
Review URL: https://codereview.chromium.org/18190006
git-svn-id: http://skia.googlecode.com/svn/trunk@10049 2bbb7eff-a529-9590-31e7-b0007b416f81
Diffstat (limited to 'tools')
-rwxr-xr-x | tools/submit_try | 11 |
1 files changed, 9 insertions, 2 deletions
diff --git a/tools/submit_try b/tools/submit_try index 4f4fd6e8a0..d135675797 100755 --- a/tools/submit_try +++ b/tools/submit_try @@ -205,9 +205,16 @@ Can also use the following aliases to run on groups of builders- Error('Cannot specify "%s" with additional builder names or ' 'aliases.' % bot) if bot == ALL_BUILDERS: - using_bots = trybots + are_you_sure = raw_input('Running a try on every bot is very ' + 'expensive. You may be able to get ' + 'enough information by running on a ' + 'smaller set of bots. Are you sure you ' + 'want to run your try job on all of the ' + 'trybots? [y,n]: ') + if are_you_sure == 'y': + using_bots = trybots elif bot == COMPILE_BUILDERS: - using_bots = [t for t in trybots if '_Compile_' in t] + using_bots = [t for t in trybots if t.startswith('Build')] elif bot == CQ_BUILDERS: using_bots = RetrieveTrybotList(json_filename='cqtrybots') elif bot == REGEX: |