[Findit] Adding variable builder to cover 'Blimp Linux (dbg)' on main waterfall

[Findit] Adding variable builder to cover 'Blimp Linux (dbg)' on main waterfall

BUG=620094
Committed: https://chromium.googlesource.com/chromium/tools/build/+/683075656e5c4eca59f224ff65e0dd36e10c918b

[lijeffrey, 2016-06-14 23:45:22]

Description was changed from

==========
[Findit] Adding variable builder to cover 'Blimp Linux (dbg)' on main waterfall

BUG=
==========

to

==========
[Findit] Adding variable builder to cover 'Blimp Linux (dbg)' on main waterfall

BUG=620094
==========

[lijeffrey, 2016-06-14 23:45:22]

lijeffrey@chromium.org changed reviewers:
+ sergeyberezin@chromium.org

[lijeffrey, 2016-06-14 23:47:36]

Hi Sergey,

This change adds another variable builder to correspond to 'Blimp Linux (dbg)'
on the main waterfall. I saw you recently made a change to these files to split
the optional slaves due to better balance disk usage, can you please have a look
that this change is ok?

Thanks,
Jeff

[Sergey Berezin, 2016-06-15 19:29:57]

LGTM + comment - check the graphs go/prhnj and see which pool makes more sense.
Thanks!

https://codereview.chromium.org/2065933003/diff/1/masters/master.tryserver.ch...
File masters/master.tryserver.chromium.linux/slaves.cfg (right):

https://codereview.chromium.org/2065933003/diff/1/masters/master.tryserver.ch...
masters/master.tryserver.chromium.linux/slaves.cfg:75:
'linux_chromium_blimp_variable',          # linux_blimp
How big do you expect linux_blimp to be?

It appears, this pool has some slaves at 75% disk capacity, while
linux_optional1 still at max 46%. If it's comparable to the full chromium
checkout in size, it might make sense to add it to optional_builders1 pool
instead (and update the comment).

[lijeffrey, 2016-06-15 21:35:27]

https://codereview.chromium.org/2065933003/diff/1/masters/master.tryserver.ch...
File masters/master.tryserver.chromium.linux/slaves.cfg (right):

https://codereview.chromium.org/2065933003/diff/1/masters/master.tryserver.ch...
masters/master.tryserver.chromium.linux/slaves.cfg:75:
'linux_chromium_blimp_variable',          # linux_blimp
On 2016/06/15 19:29:57, Sergey Berezin wrote:
> How big do you expect linux_blimp to be?
> 
> It appears, this pool has some slaves at 75% disk capacity, while
> linux_optional1 still at max 46%. If it's comparable to the full chromium
> checkout in size, it might make sense to add it to optional_builders1 pool
> instead (and update the comment).

I spoke with haibinlu@ and blimp is just a content shell so should be smaller
than full chromium, so this pool should be ok for now :)

[lijeffrey, 2016-06-15 21:35:35]

The CQ bit was checked by lijeffrey@chromium.org

[commit-bot: I haz the power, 2016-06-15 21:35:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2065933003/1

[commit-bot: I haz the power, 2016-06-15 21:38:41]

Description was changed from

==========
[Findit] Adding variable builder to cover 'Blimp Linux (dbg)' on main waterfall

BUG=620094
==========

to

==========
[Findit] Adding variable builder to cover 'Blimp Linux (dbg)' on main waterfall

BUG=620094

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

[commit-bot: I haz the power, 2016-06-15 21:38:42]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/tools/build/+/683075656e5c4eca59f2...