Adding Findit high-end trybots to machines that can use higher -j value for compile

Adding Findit high-end trybots to machines that can use higher -j value for compile

Findit has dedicated buildslaves spec'ed higher than normal (32 cores as opposed to 8), namely:

slave799-c4
slave873-c4
slave874-c4
slave875-c4
slave876-c4
slave877-c4

Because these bots have higher cpu counts, they can take advantage of an increased number of parallel tasks instead of the default 50.

BUG=666802
Committed: https://chromium.googlesource.com/chromium/tools/build/+/f33bbbd7b7f71ef15a38f9c85d70d47b8358d151

[lijeffrey, 2016-11-18 18:42:58]

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

[lijeffrey, 2016-11-18 18:42:59]

Hi Ukai,

Thanks again for helping us look into the performance for our super bots! This
change adds Findit's bots to the list that can use 10 x #cpus, can you please
have a look, or help forward to the appropriate owner?

Thanks!
Jeff

[lijeffrey, 2016-11-18 18:46:13]

Description was changed from

==========
Adding Findit high-end trybots to machines that can use higher -j value for
compile

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

to

==========
Adding Findit high-end trybots to machines that can use higher -j value for
compile

Findit has dedicated buildslaves spec'ed higher than normal (32 cores as opposed
to 8), namely:

slave799-c4
slave873-c4
slave874-c4
slave875-c4
slave876-c4
slave877-c4

Because these bots have higher cpu counts, they can take advantage of an
increased number of parallel tasks instead of the default 50.

BUG=666802
==========

[ukai, 2016-11-21 01:39:54]

lgtm

[stgao, 2016-11-21 02:52:09]

The CQ bit was checked by stgao@chromium.org

[commit-bot: I haz the power, 2016-11-21 02:52:13]

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-21 03:29:32]

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

[commit-bot: I haz the power, 2016-11-21 03:29:33]

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

[lijeffrey, 2016-11-21 17:33:09]

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

[lijeffrey, 2016-11-21 17:33:54]

Hi Sergey, it seems we need an lg2m from someone in OWNERS to get this
committed. Can you ptal?

Thanks,
Jeff

[Sergey Berezin, 2016-11-21 20:34:38]

LGTM for this specific change.

As an aside (and not to block this change), it's odd that we hard-code specific
slaves into this code. Ideally, specific machine names and their custom limits
would be a part of some configuration.

[lijeffrey, 2016-11-21 20:35:21]

The CQ bit was checked by lijeffrey@chromium.org

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

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-21 20:45:02]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1479760521781300, "parent_rev":
"953d4bca1510c987123b73c3a97d7ad12c1eae39", "commit_rev":
"f33bbbd7b7f71ef15a38f9c85d70d47b8358d151"}

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

Description was changed from

==========
Adding Findit high-end trybots to machines that can use higher -j value for
compile

Findit has dedicated buildslaves spec'ed higher than normal (32 cores as opposed
to 8), namely:

slave799-c4
slave873-c4
slave874-c4
slave875-c4
slave876-c4
slave877-c4

Because these bots have higher cpu counts, they can take advantage of an
increased number of parallel tasks instead of the default 50.

BUG=666802
==========

to

==========
Adding Findit high-end trybots to machines that can use higher -j value for
compile

Findit has dedicated buildslaves spec'ed higher than normal (32 cores as opposed
to 8), namely:

slave799-c4
slave873-c4
slave874-c4
slave875-c4
slave876-c4
slave877-c4

Because these bots have higher cpu counts, they can take advantage of an
increased number of parallel tasks instead of the default 50.

BUG=666802

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

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

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