In rebaseline-cl, trigger try jobs with builder per invocation of git cl try.

In rebaseline-cl, trigger try jobs with builder per invocation of git cl try.

If android_blink_rel is included in the list of try bots, then using
git cl try with all of the try bots including that one will fail.

Some alternative possible solutions:
 - Make an explicit special case for android_blink_rel,
   starting that one in a separate command.
 - Remove android_blink_rel from the list of builders again.
 - Remove android_blink_rel only from the list of builders
   triggered here.

Committed: https://crrev.com/44ce301713688bb031e2b3b45bae7071aae71d57
Cr-Commit-Position: refs/heads/master@{#414152}

[qyearsley, 2016-08-23 21:01:28]

qyearsley@chromium.org changed reviewers:
+ dpranke@chromium.org, wkorman@chromium.org

[wkorman, 2016-08-23 23:30:46]

https://codereview.chromium.org/2269253002/diff/1/third_party/WebKit/Tools/Sc...
File third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_cl.py
(right):

https://codereview.chromium.org/2269253002/diff/1/third_party/WebKit/Tools/Sc...
third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_cl.py:106:
_log.info(git_cl.run(['try', '-b', builder]))
remove log wrapper, looks like debugging?

[qyearsley, 2016-08-24 18:17:44]

https://codereview.chromium.org/2269253002/diff/1/third_party/WebKit/Tools/Sc...
File third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_cl.py
(right):

https://codereview.chromium.org/2269253002/diff/1/third_party/WebKit/Tools/Sc...
third_party/WebKit/Tools/Scripts/webkitpy/tool/commands/rebaseline_cl.py:106:
_log.info(git_cl.run(['try', '-b', builder]))
On 2016/08/23 at 23:30:46, wkorman wrote:
> remove log wrapper, looks like debugging?

I originally intended for this to provide more output to the user about what git
cl try is doing.

Although the list of builders that jobs are triggered for is already printed
above, and it's a good idea to not print more than required and not print
redundant information. So, I removed that in the latest patchset.

[qyearsley, 2016-08-24 18:17:51]

The CQ bit was checked by qyearsley@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-24 18:18:42]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Dirk Pranke, 2016-08-24 18:22:23]

lgtm. 

It would be good to fix this in git-cl try.

[commit-bot: I haz the power, 2016-08-24 19:40:32]

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

[commit-bot: I haz the power, 2016-08-24 19:40:33]

Dry run: This issue passed the CQ dry run.

[qyearsley, 2016-08-24 20:06:43]

The CQ bit was checked by qyearsley@chromium.org

[qyearsley, 2016-08-24 20:06:44]

The patchset sent to the CQ was uploaded after l-g-t-m from dpranke@chromium.org
Link to the patchset: https://codereview.chromium.org/2269253002/#ps40001
(title: "Rebased")

[commit-bot: I haz the power, 2016-08-24 20:07:27]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[qyearsley, 2016-08-24 20:54:52]

On 2016/08/24 at 18:22:23, dpranke wrote:
> lgtm. 
> 
> It would be good to fix this in git-cl try.

Agreed -- filed bug http://crbug.com/640740.

[commit-bot: I haz the power, 2016-08-24 21:34:29]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-24 21:36:23]

Description was changed from

==========
In rebaseline-cl, trigger try jobs with builder per invocation of git cl try.

If android_blink_rel is included in the list of try bots, then using
git cl try with all of the try bots including that one will fail.

Some alternative possible solutions:
 - Make an explicit special case for android_blink_rel,
   starting that one in a separate command.
 - Remove android_blink_rel from the list of builders again.
 - Remove android_blink_rel only from the list of builders
   triggered here.
==========

to

==========
In rebaseline-cl, trigger try jobs with builder per invocation of git cl try.

If android_blink_rel is included in the list of try bots, then using
git cl try with all of the try bots including that one will fail.

Some alternative possible solutions:
 - Make an explicit special case for android_blink_rel,
   starting that one in a separate command.
 - Remove android_blink_rel from the list of builders again.
 - Remove android_blink_rel only from the list of builders
   triggered here.

Committed: https://crrev.com/44ce301713688bb031e2b3b45bae7071aae71d57
Cr-Commit-Position: refs/heads/master@{#414152}
==========

[commit-bot: I haz the power, 2016-08-24 21:36:24]

Patchset 3 (id:??) landed as
https://crrev.com/44ce301713688bb031e2b3b45bae7071aae71d57
Cr-Commit-Position: refs/heads/master@{#414152}