Move some non-tree closing bots on chromium.android to chromium.android.fyi

Move some non-tree closing bots on chromium.android to chromium.android.fyi

These are the builders with about 50% pass rate.

BUG=660180
Committed: https://chromium.googlesource.com/chromium/tools/build/+/254b63d101d5e09990d0d969dcb426af6e0dcf6e

[Ryan Tseng, 2016-11-04 22:32:28]

The CQ bit was checked by hinoka@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-11-04 22:32:38]

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-11-04 22:46:47]

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

[commit-bot: I haz the power, 2016-11-04 22:46:48]

Dry run: Try jobs failed on following builders:
  Build Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/324a5636a5cfdf10)

[hinoka, 2016-11-04 23:02:36]

The CQ bit was checked by hinoka@chromium.org

[commit-bot: I haz the power, 2016-11-04 23:02:39]

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-11-04 23:02:40]

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

[commit-bot: I haz the power, 2016-11-04 23:02:41]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
Committers are members of the group "project-infra-committers".
Note that this has nothing to do with OWNERS files.

[hinoka, 2016-11-04 23:25:50]

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

[commit-bot: I haz the power, 2016-11-04 23:25:57]

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

[hinoka, 2016-11-04 23:26:21]

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

[hinoka, 2016-11-04 23:26:21]

ptal.  Namely, see if the builders being moved over are the right ones.

[jbudorick, 2016-11-04 23:30:04]

https://codereview.chromium.org/2478413002/diff/1/masters/master.chromium.and...
File masters/master.chromium.android.fyi/builders.pyl (right):

https://codereview.chromium.org/2478413002/diff/1/masters/master.chromium.and...
masters/master.chromium.android.fyi/builders.pyl:14: "builders": {
This should only move:
 - L Consumer
 - L Low-end

[hinoka, 2016-11-04 23:43:45]

https://codereview.chromium.org/2478413002/diff/1/masters/master.chromium.and...
File masters/master.chromium.android.fyi/builders.pyl (right):

https://codereview.chromium.org/2478413002/diff/1/masters/master.chromium.and...
masters/master.chromium.android.fyi/builders.pyl:14: "builders": {
On 2016/11/04 23:30:04, jbudorick wrote:
> This should only move:
>  - L Consumer
>  - L Low-end

Only two? The rest of them look pretty red to me too.

[jbudorick, 2016-11-04 23:49:35]

On 2016/11/04 23:43:45, hinoka wrote:
>
https://codereview.chromium.org/2478413002/diff/1/masters/master.chromium.and...
> File masters/master.chromium.android.fyi/builders.pyl (right):
> 
>
https://codereview.chromium.org/2478413002/diff/1/masters/master.chromium.and...
> masters/master.chromium.android.fyi/builders.pyl:14: "builders": {
> On 2016/11/04 23:30:04, jbudorick wrote:
> > This should only move:
> >  - L Consumer
> >  - L Low-end
> 
> Only two? The rest of them look pretty red to me too.

They are, but they need to be fixed rather than FYI'ed.

[Ryan Tseng, 2016-11-04 23:52:49]

The CQ bit was checked by hinoka@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-11-04 23:52:59]

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-11-05 00:34:33]

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

[commit-bot: I haz the power, 2016-11-05 00:34:34]

Dry run: Try jobs failed on following builders:
  Build Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/324ab0ee3849db10)

[jbudorick, 2016-11-09 20:13:57]

On 2016/11/04 23:49:35, jbudorick wrote:
> On 2016/11/04 23:43:45, hinoka wrote:
> >
>
https://codereview.chromium.org/2478413002/diff/1/masters/master.chromium.and...
> > File masters/master.chromium.android.fyi/builders.pyl (right):
> > 
> >
>
https://codereview.chromium.org/2478413002/diff/1/masters/master.chromium.and...
> > masters/master.chromium.android.fyi/builders.pyl:14: "builders": {
> > On 2016/11/04 23:30:04, jbudorick wrote:
> > > This should only move:
> > >  - L Consumer
> > >  - L Low-end
> > 
> > Only two? The rest of them look pretty red to me too.
> 
> They are, but they need to be fixed rather than FYI'ed.

As Ben pointed out, JB should move over to FYI, too.

[Ryan Tseng, 2016-11-09 20:30:34]

The CQ bit was checked by hinoka@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-11-09 20:30:39]

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

[Ryan Tseng, 2016-11-09 20:33:22]

The CQ bit was checked by hinoka@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-11-09 20:33:28]

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

[Ryan Tseng, 2016-11-09 20:57:09]

The CQ bit was checked by hinoka@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-11-09 20:57:11]

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-11-09 21:34:24]

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

[commit-bot: I haz the power, 2016-11-09 21:34:25]

Dry run: Try jobs failed on following builders:
  Build Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/3263b2fbecb15610)

[Ryan Tseng, 2016-11-09 22:43:16]

The CQ bit was checked by hinoka@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-11-09 22:43:24]

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-11-09 22:54:14]

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

[commit-bot: I haz the power, 2016-11-09 22:54:15]

Dry run: This issue passed the CQ dry run.

[hinoka, 2016-11-09 22:59:43]

ptal again

there will be a separate CL to get the triggering to work.  Until then the FYI
builders will be red.

[bpastene, 2016-11-10 01:38:51]

Is there a plan wrt all the needed recipe changes? The bots will also need to be
removed from
https://chromium.googlesource.com/chromium/tools/build/+/master/scripts/slave...
prior to master restart.

https://codereview.chromium.org/2478413002/diff/100001/masters/master.chromiu...
File masters/master.chromium.android/builders.pyl (right):

https://codereview.chromium.org/2478413002/diff/100001/masters/master.chromiu...
masters/master.chromium.android/builders.pyl:15: "Lollipop Tablet Tester": {
nit: Could you re-alphabetize these please?

[Ryan Tseng, 2016-11-11 20:33:53]

rebase

[Ryan Tseng, 2016-11-11 20:33:53]

The CQ bit was checked by hinoka@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-11-11 20:34:00]

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

[hinoka, 2016-11-11 20:34:53]

Recipe changes here: https://codereview.chromium.org/2491033005/

Seperate becase they have to be committed at master restart time instead

[Ryan Tseng, 2016-11-11 20:37:26]

The CQ bit was checked by hinoka@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-11-11 20:37:31]

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

[hinoka, 2016-11-11 20:39:14]

ptal now

https://codereview.chromium.org/2478413002/diff/100001/masters/master.chromiu...
File masters/master.chromium.android/builders.pyl (right):

https://codereview.chromium.org/2478413002/diff/100001/masters/master.chromiu...
masters/master.chromium.android/builders.pyl:15: "Lollipop Tablet Tester": {
On 2016/11/10 01:38:50, bpastene wrote:
> nit: Could you re-alphabetize these please?

Done.

[commit-bot: I haz the power, 2016-11-11 20:48:21]

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

[commit-bot: I haz the power, 2016-11-11 20:48:22]

Dry run: This issue passed the CQ dry run.

[ghost stip (do not use), 2016-11-11 21:34:02]

lgtm

[hinoka, 2016-11-11 21:35:22]

The CQ bit was checked by hinoka@chromium.org

[commit-bot: I haz the power, 2016-11-11 21:35:26]

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-11-11 21:46:27]

Description was changed from

==========
Move some non-tree closing bots on chromium.android to chromium.android.fyi

These are the builders with about 50% pass rate.

BUG=660180
==========

to

==========
Move some non-tree closing bots on chromium.android to chromium.android.fyi

These are the builders with about 50% pass rate.

BUG=660180

Committed:
https://chromium.googlesource.com/chromium/tools/build/+/254b63d101d5e09990d0...
==========

[commit-bot: I haz the power, 2016-11-11 21:46:28]

Committed patchset #8 (id:140001) as
https://chromium.googlesource.com/chromium/tools/build/+/254b63d101d5e09990d0...