Changed the step name for wait_for_any.py to include only one revision.

Changed the step name for wait_for_any.py to include only one revision.

If there are more than one, then just say how many more. This prevents
us from logging a step name longer than buildbot supports.

R=qyearsley@chromium.org

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=294677

[qyearsley, 2015-04-02 04:08:49]

LGTM

https://codereview.chromium.org/1050253003/diff/1/scripts/slave/recipe_module...
File scripts/slave/recipe_modules/auto_bisect/bisector.py (right):

https://codereview.chromium.org/1050253003/diff/1/scripts/slave/recipe_module...
scripts/slave/recipe_modules/auto_bisect/bisector.py:401: name = 'Waiting for
revision: ' + revision_list[0].revision_string
Very unimportant nit: the name might read more smoothly without the colon IMO,
e.g. "Waiting for revision 300300 and 2 more revision(s)."

[RobertoCN, 2015-04-03 03:35:41]

The CQ bit was checked by robertocn@chromium.org

[RobertoCN, 2015-04-03 03:35:41]

The patchset sent to the CQ was uploaded after l-g-t-m from
qyearsley@chromium.org
Link to the patchset: https://codereview.chromium.org/1050253003/#ps20001
(title: "Removing colon.")

[commit-bot: I haz the power, 2015-04-03 03:36:51]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1050253003/20001

[commit-bot: I haz the power, 2015-04-03 03:39:13]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-04-03 03:39:13]

Presubmit check for 1050253003-20001 failed and returned exit status 1.

Running presubmit commit checks ...

** Presubmit ERRORS **
Missing LGTM from an OWNER for these files:
    build/scripts/slave/recipes/bisect.expected/basic.json
    build/scripts/slave/recipes/bisect.expected/encoded_config_test.json

Presubmit checks took 133.9s to calculate.

Was the presubmit check useful? If not, run "git cl presubmit -v"
to figure out which PRESUBMIT.py was run, then run git blame
on the file to figure out who to ask for help.

[qyearsley, 2015-04-03 17:08:35]

qyearsley@chromium.org changed reviewers:
+ iannucci@chromium.org

[qyearsley, 2015-04-03 17:09:50]

Needs OWNER LGTM.

Robbie, should Roberto be added as OWNER of recipes/bisect.expected? If so, it
can be added in https://codereview.chromium.org/1049833002/.

On 2015/04/03 03:39:13, I haz the power (commit-bot) wrote:
> Presubmit check for 1050253003-20001 failed and returned exit status 1.
> 
> Running presubmit commit checks ...
> 
> ** Presubmit ERRORS **
> Missing LGTM from an OWNER for these files:
>     build/scripts/slave/recipes/bisect.expected/basic.json
>     build/scripts/slave/recipes/bisect.expected/encoded_config_test.json
> 
> Presubmit checks took 133.9s to calculate.
> 
> Was the presubmit check useful? If not, run "git cl presubmit -v"
> to figure out which PRESUBMIT.py was run, then run git blame
> on the file to figure out who to ask for help.

[RobertoCN, 2015-04-03 19:05:09]

https://codereview.chromium.org/1050253003/diff/1/scripts/slave/recipe_module...
File scripts/slave/recipe_modules/auto_bisect/bisector.py (right):

https://codereview.chromium.org/1050253003/diff/1/scripts/slave/recipe_module...
scripts/slave/recipe_modules/auto_bisect/bisector.py:401: name = 'Waiting for
revision: ' + revision_list[0].revision_string
On 2015/04/02 04:08:49, qyearsley wrote:
> Very unimportant nit: the name might read more smoothly without the colon IMO,
> e.g. "Waiting for revision 300300 and 2 more revision(s)."

Done.

[iannucci, 2015-04-03 20:49:25]

On 2015/04/03 at 19:05:09, robertocn wrote:
>
https://codereview.chromium.org/1050253003/diff/1/scripts/slave/recipe_module...
> File scripts/slave/recipe_modules/auto_bisect/bisector.py (right):
> 
>
https://codereview.chromium.org/1050253003/diff/1/scripts/slave/recipe_module...
> scripts/slave/recipe_modules/auto_bisect/bisector.py:401: name = 'Waiting for
revision: ' + revision_list[0].revision_string
> On 2015/04/02 04:08:49, qyearsley wrote:
> > Very unimportant nit: the name might read more smoothly without the colon
IMO,
> > e.g. "Waiting for revision 300300 and 2 more revision(s)."
> 
> Done.

Oh... You folks should be OWNERS for this module+recipe. Please add yourselves
and send me the CL for that.

lgtm.

[RobertoCN, 2015-04-03 20:50:36]

The CQ bit was checked by robertocn@chromium.org

[commit-bot: I haz the power, 2015-04-03 20:50:52]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1050253003/20001

[commit-bot: I haz the power, 2015-04-03 20:53:34]

Committed patchset #2 (id:20001) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=294677