[Android] Set up continuous builder + trybot for cronet.

[Android] Set up continuous builder + trybot for cronet.

Uses the chromium recipe, unlike the other cronet bots.

BUG=634350
Committed: https://chromium.googlesource.com/chromium/tools/build/+/239067c0b2b9d9f905d8a0925cd13439edead8ae

[jbudorick, 2016-09-02 18:05:36]

jbudorick@chromium.org changed reviewers:
+ bpastene@chromium.org, stip@chromium.org, xunjieli@chromium.org

[xunjieli, 2016-09-05 23:49:35]

Really look forward to seeing the bot online! I don't see anything that I can
comment on. Deferring this to infra owners. Thanks a lot.

[ghost stip (do not use), 2016-09-06 23:05:35]

lgtm

[bpastene, 2016-09-06 23:29:27]

lgtm + comments

https://codereview.chromium.org/2308683002/diff/1/masters/master.tryserver.ch...
File masters/master.tryserver.chromium.android/builders.pyl (right):

https://codereview.chromium.org/2308683002/diff/1/masters/master.tryserver.ch...
masters/master.tryserver.chromium.android/builders.pyl:112: "slave_pools":
["ccompute_bots_cq"],
If this is going to be a full CQ builder, we may need to increase the size of
that pool. If it's just an optional trybot, could we use
"ccompute_optional_bots" instead?

https://codereview.chromium.org/2308683002/diff/1/scripts/slave/recipe_module...
File scripts/slave/recipe_modules/chromium_tests/chromium_android.py (right):

https://codereview.chromium.org/2308683002/diff/1/scripts/slave/recipe_module...
scripts/slave/recipe_modules/chromium_tests/chromium_android.py:56:
'android_config': 'main_builder_rel_mb',
mb_config.pyl change?

[jbudorick, 2016-09-06 23:31:07]

https://codereview.chromium.org/2308683002/diff/1/masters/master.tryserver.ch...
File masters/master.tryserver.chromium.android/builders.pyl (right):

https://codereview.chromium.org/2308683002/diff/1/masters/master.tryserver.ch...
masters/master.tryserver.chromium.android/builders.pyl:112: "slave_pools":
["ccompute_bots_cq"],
On 2016/09/06 23:29:26, bpastene wrote:
> If this is going to be a full CQ builder, we may need to increase the size of
> that pool. If it's just an optional trybot, could we use
> "ccompute_optional_bots" instead?

It will be a full CQ builder eventually. I'm planning to start it as a low
percentage experiment.

https://codereview.chromium.org/2308683002/diff/1/scripts/slave/recipe_module...
File scripts/slave/recipe_modules/chromium_tests/chromium_android.py (right):

https://codereview.chromium.org/2308683002/diff/1/scripts/slave/recipe_module...
scripts/slave/recipe_modules/chromium_tests/chromium_android.py:56:
'android_config': 'main_builder_rel_mb',
On 2016/09/06 23:29:26, bpastene wrote:
> mb_config.pyl change?

https://codereview.chromium.org/2302353003/

[jbudorick, 2016-09-07 20:44:40]

The CQ bit was checked by jbudorick@chromium.org

[jbudorick, 2016-09-07 20:44:40]

The patchset sent to the CQ was uploaded after l-g-t-m from
bpastene@chromium.org, stip@chromium.org
Link to the patchset: https://codereview.chromium.org/2308683002/#ps20001
(title: "update expectations")

[commit-bot: I haz the power, 2016-09-07 20:44:48]

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-07 20:48:44]

Description was changed from

==========
[Android] Set up continuous builder + trybot for cronet.

Uses the chromium recipe, unlike the other cronet bots.

BUG=634350
==========

to

==========
[Android] Set up continuous builder + trybot for cronet.

Uses the chromium recipe, unlike the other cronet bots.

BUG=634350

Committed:
https://chromium.googlesource.com/chromium/tools/build/+/239067c0b2b9d9f905d8...
==========

[commit-bot: I haz the power, 2016-09-07 20:48:45]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/tools/build/+/239067c0b2b9d9f905d8...