Add a buildbotless Android bot

Add a buildbotless Android bot

BUG=skia:5626
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2428603002

Committed: https://skia.googlesource.com/skia/+/5238343ab27a9f42983b6c79f199416a5ae58542

[borenet2, 2016-10-17 15:18:47]

Description was changed from

==========
Add a buildbotless Android bot

BUG=skia:5626
==========

to

==========
Add a buildbotless Android bot

BUG=skia:5626
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2428603002
==========

[borenet, 2016-10-17 16:09:54]

borenet@google.com changed reviewers:
+ benjaminwagner@google.com, borenet@google.com

[mtklein, 2016-10-17 16:32:01]

mtklein@google.com changed reviewers:
+ mtklein@google.com

[mtklein, 2016-10-17 16:32:02]

*\o/*

[dogben, 2016-10-17 16:32:31]

benjaminwagner@google.com changed reviewers:
- mtklein@google.com

[dogben, 2016-10-17 16:32:31]

lgtm

https://codereview.chromium.org/2428603002/diff/20001/infra/bots/gen_tasks.go
File infra/bots/gen_tasks.go (right):

https://codereview.chromium.org/2428603002/diff/20001/infra/bots/gen_tasks.go...
infra/bots/gen_tasks.go:485: if parts["role"] != "Housekeeper" {
Should this just be name != "Housekeeper-PerCommit-InfraTests" and can be
permanent?

Alternatively, should test(), perf(), and housekeeper() be the ones to call
compile()?

[borenet, 2016-10-17 16:51:03]

https://codereview.chromium.org/2428603002/diff/20001/infra/bots/gen_tasks.go
File infra/bots/gen_tasks.go (right):

https://codereview.chromium.org/2428603002/diff/20001/infra/bots/gen_tasks.go...
infra/bots/gen_tasks.go:485: if parts["role"] != "Housekeeper" {
On 2016/10/17 16:32:31, dogben wrote:
> Should this just be name != "Housekeeper-PerCommit-InfraTests" and can be
> permanent?
> 
> Alternatively, should test(), perf(), and housekeeper() be the ones to call
> compile()?

So we need this if-statement just to prevent us from creating an orphaned
compile task for the Housekeeper-PerCommit bot, which doesn't yet exist.  Once
we add that bot, we can remove this and everything works fine.  As for moving
the call to compile, I'm fine with that in the long run, but for now I'd like to
match the flow in the swarm_trigger recipe as closely as possible.

[borenet, 2016-10-17 16:52:14]

The CQ bit was checked by borenet@google.com

[commit-bot: I haz the power, 2016-10-17 16:52:22]

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-10-17 17:17:55]

Description was changed from

==========
Add a buildbotless Android bot

BUG=skia:5626
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2428603002
==========

to

==========
Add a buildbotless Android bot

BUG=skia:5626
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2428603002

Committed:
https://skia.googlesource.com/skia/+/5238343ab27a9f42983b6c79f199416a5ae58542
==========

[commit-bot: I haz the power, 2016-10-17 17:17:56]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/5238343ab27a9f42983b6c79f199416a5ae58542