Move memory FYI slaves to memory full master and switch configs over.

Move memory FYI slaves to memory full master and switch configs over.

After this has been applied,  the full master will be the real, running
master and the fyi master can be removed.

This also makes the memory full master a tree closer, as the point of
this change is that it isn't a non-closer / fyi only master.

BUG=597178
Committed: https://chromium.googlesource.com/chromium/tools/build/+/36281a7d0d221c037e3e245e0de0a79cba6f581f

[benwells, 2016-07-28 08:14:14]

benwells@chromium.org changed reviewers:
+ dpranke@chromium.org, vadimsh@chromium.org

[benwells, 2016-07-28 08:14:15]

https://codereview.chromium.org/2184603003/diff/20001/scripts/slave/gatekeepe...
File scripts/slave/gatekeeper_trees.json (left):

https://codereview.chromium.org/2184603003/diff/20001/scripts/slave/gatekeepe...
scripts/slave/gatekeeper_trees.json:144:
"https://build.chromium.org/p/chromium.memory.fyi": ["*"],
This is deliberately making chromium.memory.full a closer (that's the point of
making it .full, not .fyi).

[Dirk Pranke, 2016-07-28 15:28:58]

lgtm. 

This is probably way too late, but is there a reason not to just merge this w/
the chromium.memory waterfall, if we want them both to be tree-closers and
handled by the main sheriffs?

[Vadim Sh., 2016-07-28 16:41:55]

vadimsh@chromium.org changed reviewers:
+ sergeyberezin@google.com

[Vadim Sh., 2016-07-28 16:41:56]

lgtm

Also adding today's trooper. Landing this will probably require restarting all
relevant buildbot slaves, so they reconnect to the new master.

[Sergey Berezin, 2016-07-28 18:20:39]

sergeyberezin@chromium.org changed reviewers:
+ sergeyberezin@chromium.org
- sergeyberezin@google.com

[Sergey Berezin, 2016-07-28 18:20:39]

If slaves need a restart, please file a bug at g.co/bugatrooper . Thanks!

[benwells, 2016-07-29 02:00:54]

On 2016/07/28 15:28:58, Dirk Pranke wrote:
> lgtm. 
> 
> This is probably way too late, but is there a reason not to just merge this w/
> the chromium.memory waterfall, if we want them both to be tree-closers and
> handled by the main sheriffs?

I've just assumed it was for page layout purposes?

[benwells, 2016-07-29 02:01:01]

The CQ bit was checked by benwells@chromium.org

[commit-bot: I haz the power, 2016-07-29 02:01:10]

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-07-29 02:06:49]

Description was changed from

==========
Move memory FYI slaves to memory full master and switch configs over.

After this has been applied,  the full master will be the real, running
master and the fyi master can be removed.

This also makes the memory full master a tree closer, as the point of
this change is that it isn't a non-closer / fyi only master.

BUG=597178
==========

to

==========
Move memory FYI slaves to memory full master and switch configs over.

After this has been applied,  the full master will be the real, running
master and the fyi master can be removed.

This also makes the memory full master a tree closer, as the point of
this change is that it isn't a non-closer / fyi only master.

BUG=597178

Committed:
https://chromium.googlesource.com/chromium/tools/build/+/36281a7d0d221c037e3e...
==========

[commit-bot: I haz the power, 2016-07-29 02:06:50]

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