Moving Findit variable builders to new high-end cpu slave

Moving Findit variable builders to new high-end cpu slave

BUG=620207
Committed: https://chromium.googlesource.com/chromium/tools/build/+/aeb18b9f5f758b6b8aefe411cb84f6819ab50930

[lijeffrey, 2016-06-15 05:52:41]

Description was changed from

==========
Moving Findit variable builders to new high-end cpu slave

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

to

==========
Moving Findit variable builders to new high-end cpu slave

BUG=https://codereview.chromium.org/2066013003
==========

[lijeffrey, 2016-06-15 05:52:41]

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

[lijeffrey, 2016-06-15 05:53:01]

Description was changed from

==========
Moving Findit variable builders to new high-end cpu slave

BUG=https://codereview.chromium.org/2066013003
==========

to

==========
Moving Findit variable builders to new high-end cpu slave

BUG=620207
==========

[lijeffrey, 2016-06-15 05:55:16]

ptal

Thanks Ryan for bot 799! This change moves one of the variable builders (with 5
main waterfall builders referencing it for try jobs) to this newly added beefy
buildslave. We'd like to monitor if there's any noticeable improvement in try
job execution time, currently at ~5 minutes per revision for compile and ~7-8
minutes per revision for test try jobs triggered in the last month against these
builders with normal bots.

[stgao, 2016-06-15 06:12:39]

stgao@chromium.org changed reviewers:
+ stgao@chromium.org

[stgao, 2016-06-15 06:12:39]

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

https://codereview.chromium.org/2066013003/diff/1/masters/master.tryserver.ch...
masters/master.tryserver.chromium.linux/slaves.cfg:101: optional_highcpu_slaves
= [799]
I guess we want to dedicate this slave to Findit at least for now.
It might worth a comment for this to avoid misuse by someone else later.

[lijeffrey, 2016-06-15 18:10:40]

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

https://codereview.chromium.org/2066013003/diff/1/masters/master.tryserver.ch...
masters/master.tryserver.chromium.linux/slaves.cfg:101: optional_highcpu_slaves
= [799]
On 2016/06/15 06:12:39, stgao wrote:
> I guess we want to dedicate this slave to Findit at least for now.
> It might worth a comment for this to avoid misuse by someone else later.

Done.

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

lgtm

[lijeffrey, 2016-06-15 20:40:18]

The CQ bit was checked by lijeffrey@chromium.org

[commit-bot: I haz the power, 2016-06-15 20:40:29]

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

[commit-bot: I haz the power, 2016-06-15 20:43:45]

Description was changed from

==========
Moving Findit variable builders to new high-end cpu slave

BUG=620207
==========

to

==========
Moving Findit variable builders to new high-end cpu slave

BUG=620207

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

[commit-bot: I haz the power, 2016-06-15 20:43:46]

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