WebRTC: Add Linux 32-bit trybots

WebRTC: Add Linux 32-bit trybots

BUG=644167
Committed: https://chromium.googlesource.com/chromium/tools/build/+/1a139d0857463dd137cad4050852173f2f70f6bc

[kjellander_chromium, 2016-09-06 07:26:45]

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

[kjellander_chromium, 2016-09-06 07:26:46]

I realize we have a naming convention problem here, but since it's very painful
to rename trybots (CQ stops working in the transition etc), I think it's fine
with these. They shouldn't be default trybots anyway, but only used when
somebody is working on fixing a 32-bit only issue (like I'm planning to do).

[ehmaldonado_chromium, 2016-09-06 07:38:33]

On 2016/09/06 07:26:46, kjellander_chromium wrote:
> I realize we have a naming convention problem here, but since it's very
painful
> to rename trybots (CQ stops working in the transition etc), I think it's fine
> with these. They shouldn't be default trybots anyway, but only used when
> somebody is working on fixing a 32-bit only issue (like I'm planning to do).

Don't they have to be added to mb_config.pyl too?

[kjellander_chromium, 2016-09-06 07:51:48]

On 2016/09/06 07:38:33, ehmaldonado_chromium wrote:
> On 2016/09/06 07:26:46, kjellander_chromium wrote:
> > I realize we have a naming convention problem here, but since it's very
> painful
> > to rename trybots (CQ stops working in the transition etc), I think it's
fine
> > with these. They shouldn't be default trybots anyway, but only used when
> > somebody is working on fixing a 32-bit only issue (like I'm planning to do).
> 
> Don't they have to be added to mb_config.pyl too?

Thanks for reminding me. https://codereview.webrtc.org/2305373003/ fixes that.

[kjellander_chromium, 2016-09-06 07:52:05]

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

[commit-bot: I haz the power, 2016-09-06 07:52:13]

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

[commit-bot: I haz the power, 2016-09-06 07:56:03]

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

[commit-bot: I haz the power, 2016-09-06 07:56:04]

Dry run: This issue passed the CQ dry run.

[ehmaldonado_chromium, 2016-09-06 07:58:47]

On 2016/09/06 07:51:48, kjellander_chromium wrote:
> On 2016/09/06 07:38:33, ehmaldonado_chromium wrote:
> > On 2016/09/06 07:26:46, kjellander_chromium wrote:
> > > I realize we have a naming convention problem here, but since it's very
> > painful
> > > to rename trybots (CQ stops working in the transition etc), I think it's
> fine
> > > with these. They shouldn't be default trybots anyway, but only used when
> > > somebody is working on fixing a 32-bit only issue (like I'm planning to
do).
> > 
> > Don't they have to be added to mb_config.pyl too?
> 
> Thanks for reminding me. https://codereview.webrtc.org/2305373003/ fixes that.

lgtm

[kjellander_chromium, 2016-09-06 08:20:08]

The CQ bit was checked by kjellander@chromium.org

[commit-bot: I haz the power, 2016-09-06 08:20:14]

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

[commit-bot: I haz the power, 2016-09-06 08:24:10]

Description was changed from

==========
WebRTC: Add Linux 32-bit trybots

BUG=644167
==========

to

==========
WebRTC: Add Linux 32-bit trybots

BUG=644167

Committed:
https://chromium.googlesource.com/chromium/tools/build/+/1a139d0857463dd137ca...
==========

[commit-bot: I haz the power, 2016-09-06 08:24:11]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/tools/build/+/1a139d0857463dd137ca...