Change chromium.memory.fyi to chromium.memory.full

Change chromium.memory.fyi to chromium.memory.full

This master isn't actually an FYI master, it's really an overflow area
for the rest of the memory tree.

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

[benwells, 2016-05-26 06:53:41]

Description was changed from

==========
Change chromium.memory.fyi to chromium.memory.full

This master isn't actually an FYI master, it's really an overflow area
for the rest of the memory tree.

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

to

==========
Change chromium.memory.fyi to chromium.memory.full

This master isn't actually an FYI master, it's really an overflow area
for the rest of the memory tree.

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

[benwells, 2016-05-26 06:53:41]

benwells@chromium.org changed reviewers:
+ thestig@chromium.org

[benwells, 2016-05-26 06:54:35]

+thestig for a first pass. if you want to just send on to dpranke let me know...

[Lei Zhang, 2016-05-26 19:23:57]

thestig@chromium.org changed reviewers:
+ dpranke@chromium.org

[Lei Zhang, 2016-05-26 19:23:58]

+dpranke

[Lei Zhang, 2016-05-26 19:25:53]

looks ok to me

https://codereview.chromium.org/2010953002/diff/1/scripts/master/failures_not...
File scripts/master/failures_notifier.py (right):

https://codereview.chromium.org/2010953002/diff/1/scripts/master/failures_not...
scripts/master/failures_notifier.py:39: # FailuresNotifiers (e.g. "ordinary"
bots + Webkit bots on Memory Full)
We don't have webkit bots anymore.

[benwells, 2016-05-31 08:22:37]

over to dpranke@ for full review

https://codereview.chromium.org/2010953002/diff/1/scripts/master/failures_not...
File scripts/master/failures_notifier.py (right):

https://codereview.chromium.org/2010953002/diff/1/scripts/master/failures_not...
scripts/master/failures_notifier.py:39: # FailuresNotifiers (e.g. "ordinary"
bots + Webkit bots on Memory Full)
On 2016/05/26 19:25:53, Lei Zhang wrote:
> We don't have webkit bots anymore.

OK ... I changed this comment to something that hopefully makes sense.

[Dirk Pranke, 2016-05-31 20:32:51]

lgtm

[benwells, 2016-06-02 01:21:13]

On 2016/05/31 20:32:51, Dirk Pranke wrote:
> lgtm

Thanks Dirk! So do I just land this and request a master restart?

[Lei Zhang, 2016-06-02 01:24:54]

I think that's all we do. +oshima (current sheriff) FYI.

[oshima, 2016-06-02 01:27:29]

On 2016/06/02 01:24:54, Lei Zhang (OOO) wrote:
> I think that's all we do. +oshima (current sheriff) FYI.

nice, thank you for heads up!

[Dirk Pranke, 2016-06-02 01:42:10]

yup, as long as all of the slaves are auto-rebooting.

[benwells, 2016-06-02 01:52:41]

On 2016/06/02 01:42:10, Dirk Pranke wrote:
> yup, as long as all of the slaves are auto-rebooting.

OK, i'll do this now and see if anything explodes...

[benwells, 2016-06-02 01:52:47]

The CQ bit was checked by benwells@chromium.org

[commit-bot: I haz the power, 2016-06-02 01:52:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2010953002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2010953002/40001

[commit-bot: I haz the power, 2016-06-02 02:03:04]

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

[commit-bot: I haz the power, 2016-06-02 02:03:06]

Try jobs failed on following builders:
  Build Presubmit on tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Build%20Presubmit/build...)

[benwells, 2016-06-02 04:26:14]

On 2016/06/02 02:03:06, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   Build Presubmit on tryserver.infra (JOB_FAILED,
>
https://build.chromium.org/p/tryserver.infra/builders/Build%20Presubmit/build...)

dpranke: I fixed this by renaming a bunch of files
(scripts/slave/recipe_modules/chromium_tests/chromium_memory_fyi.py and
scripts/slave/recipes/chromium.expected/full_chromium_memory_fyi_*).

Please have another look.

[Dirk Pranke, 2016-06-02 17:47:51]

still lgtm.

https://codereview.chromium.org/2010953002/diff/60001/scripts/slave/recipe_mo...
File scripts/slave/recipe_modules/chromium_tests/chromium_memory_fyi.py (left):

https://codereview.chromium.org/2010953002/diff/60001/scripts/slave/recipe_mo...
scripts/slave/recipe_modules/chromium_tests/chromium_memory_fyi.py:1: #
Copyright 2014 The Chromium Authors. All rights reserved.
Hm. I wonder why this doesn't show up as "D" on Rietveld ...

[benwells, 2016-06-02 23:18:40]

The CQ bit was checked by benwells@chromium.org

[benwells, 2016-06-02 23:18:40]

The CQ bit was unchecked by benwells@chromium.org

[benwells, 2016-06-02 23:22:21]

The CQ bit was checked by benwells@chromium.org

[commit-bot: I haz the power, 2016-06-02 23:22:37]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2010953002/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2010953002/60001

[commit-bot: I haz the power, 2016-06-02 23:28:42]

Description was changed from

==========
Change chromium.memory.fyi to chromium.memory.full

This master isn't actually an FYI master, it's really an overflow area
for the rest of the memory tree.

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

to

==========
Change chromium.memory.fyi to chromium.memory.full

This master isn't actually an FYI master, it's really an overflow area
for the rest of the memory tree.

BUG=597178

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

[commit-bot: I haz the power, 2016-06-02 23:28:44]

Committed patchset #4 (id:60001) as
https://chromium.googlesource.com/chromium/tools/build/+/ec3e72dd7b0e8a080771...

[benwells, 2016-06-03 03:55:35]

A revert of this CL (patchset #4 id:60001) has been created in
https://codereview.chromium.org/2032043003/ by benwells@chromium.org.

The reason for reverting is: The master restart script had a problem with this.
Reverting and will reland once troopers have the problems figured out..