tools/roll*: use master.tryserver instead of tryserver

tools/roll*: use master.tryserver instead of tryserver

BUG=617627
TBR=kjellander@chromium.org

Committed: https://crrev.com/65854075e069afd2c5be3fbebc9c99cf36e5b59e
Cr-Commit-Position: refs/heads/master@{#405543}

[Corentin Wallez, 2016-07-14 17:35:44]

cwallez@chromium.org changed reviewers:
+ dpranke@chromium.org

[Corentin Wallez, 2016-07-14 17:36:01]

dpranke: PTAL

Others: FYI

[tandrii(chromium), 2016-07-14 17:46:56]

tandrii@chromium.org changed reviewers:
+ tandrii@chromium.org

[tandrii(chromium), 2016-07-14 17:46:57]

lgtm

[Dirk Pranke, 2016-07-14 18:31:14]

I don't know these scripts, so I defer to tandrii@ here, who hopefully does.

[tandrii(chromium), 2016-07-14 18:33:41]

On 2016/07/14 18:31:14, Dirk Pranke wrote:
> I don't know these scripts, so I defer to tandrii@ here, who hopefully does.

I don't know these scripts either, but the change here is about what gets
inserted into CQ_INCLUDE_TRYBOTS directive, which I understand, and so LGTM.

[Corentin Wallez, 2016-07-14 18:39:53]

On 2016/07/14 at 18:33:41, tandrii wrote:
> On 2016/07/14 18:31:14, Dirk Pranke wrote:
> > I don't know these scripts, so I defer to tandrii@ here, who hopefully does.
> 
> I don't know these scripts either, but the change here is about what gets
inserted into CQ_INCLUDE_TRYBOTS directive, which I understand, and so LGTM.

kjellander@ would be the best reviewer, but he is currently OoO. He is the only
per-file owner of roll_webrtc.py so this CL still needs OWNER's review.

[tandrii(chromium), 2016-07-14 18:54:31]

On 2016/07/14 18:39:53, Corentin Wallez wrote:
> On 2016/07/14 at 18:33:41, tandrii wrote:
> > On 2016/07/14 18:31:14, Dirk Pranke wrote:
> > > I don't know these scripts, so I defer to tandrii@ here, who hopefully
does.
> > 
> > I don't know these scripts either, but the change here is about what gets
> inserted into CQ_INCLUDE_TRYBOTS directive, which I understand, and so LGTM.
> 
> kjellander@ would be the best reviewer, but he is currently OoO. He is the
only
> per-file owner of roll_webrtc.py so this CL still needs OWNER's review.

How about TBR kjellander@?

[Corentin Wallez, 2016-07-14 18:55:19]

Description was changed from

==========
tools/roll*: use master.tryserver instead of tryserver

BUG=617627
==========

to

==========
tools/roll*: use master.tryserver instead of tryserver

BUG=617627
TBR=kjellander@chromium.org
==========

[Corentin Wallez, 2016-07-14 18:55:23]

The CQ bit was checked by cwallez@chromium.org

[Corentin Wallez, 2016-07-14 18:55:44]

On 2016/07/14 at 18:54:31, tandrii wrote:
> On 2016/07/14 18:39:53, Corentin Wallez wrote:
> > On 2016/07/14 at 18:33:41, tandrii wrote:
> > > On 2016/07/14 18:31:14, Dirk Pranke wrote:
> > > > I don't know these scripts, so I defer to tandrii@ here, who hopefully
does.
> > > 
> > > I don't know these scripts either, but the change here is about what gets
> > inserted into CQ_INCLUDE_TRYBOTS directive, which I understand, and so LGTM.
> > 
> > kjellander@ would be the best reviewer, but he is currently OoO. He is the
only
> > per-file owner of roll_webrtc.py so this CL still needs OWNER's review.
> 
> How about TBR kjellander@?

Done

[Dirk Pranke, 2016-07-14 18:55:57]

oh, this is in chromium.

sure, lgtm

[commit-bot: I haz the power, 2016-07-14 18:56:24]

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

[Ken Russell (switch to Gerrit), 2016-07-14 18:59:25]

kbr@chromium.org changed reviewers:
+ kbr@chromium.org

[Ken Russell (switch to Gerrit), 2016-07-14 18:59:25]

LGTM. Thanks for the cleanup.

[commit-bot: I haz the power, 2016-07-14 19:38:19]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-14 19:40:12]

Description was changed from

==========
tools/roll*: use master.tryserver instead of tryserver

BUG=617627
TBR=kjellander@chromium.org
==========

to

==========
tools/roll*: use master.tryserver instead of tryserver

BUG=617627
TBR=kjellander@chromium.org

Committed: https://crrev.com/65854075e069afd2c5be3fbebc9c99cf36e5b59e
Cr-Commit-Position: refs/heads/master@{#405543}
==========

[commit-bot: I haz the power, 2016-07-14 19:40:13]

Patchset 1 (id:??) landed as
https://crrev.com/65854075e069afd2c5be3fbebc9c99cf36e5b59e
Cr-Commit-Position: refs/heads/master@{#405543}

[kjellander_chromium, 2016-07-24 09:40:54]

kjellander@chromium.org changed reviewers:
+ kjellander@chromium.org

[kjellander_chromium, 2016-07-24 09:40:54]

lgtm