New issue
Advanced search Search tips

Issue 853050 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 13
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Request downgrade of slave270-c1 and slave339-c1 to n1-standard-8

Project Member Reported by p...@chromium.org, Jun 15 2018

Issue description

We need these machines to be downgraded to help us figure out the minimum capacity required for ThinLTO chromium.perf builders.
 

Comment 1 by fried...@google.com, Jun 15 2018

Owner: friedman@chromium.org
Status: Assigned (was: Untriaged)
If you're trying to test different configs it might be easier to just temporarily spin up one of each and let you compare.

Not that you'd need 1 of each of these but this is the general list of what I assume you're interested in testing:
NAME            CPUS  MEMORY_GB
n1-highcpu-8    8     7.20
n1-standard-8   8     30.00
n1-highmem-8    8     52.00
n1-highcpu-16   16    14.40
n1-standard-16  16    60.00
n1-highmem-16   16    104.00
n1-highcpu-32   32    28.80
n1-standard-32  32    120.00
n1-highmem-32   32    208.00

Comment 2 by p...@chromium.org, Jun 15 2018

If you could do that instead that would be very helpful.

I think we need to test just these configurations:
n1-standard-8   8     30.00
n1-highmem-8    8     52.00
n1-standard-16  16    60.00
n1-highmem-16   16    104.00
since we've already established that n1-highmem-16 would work fine.

Comment 3 by fried...@google.com, Jun 15 2018

slave{244..247}-c1

244: n1-standard-8
245: n1-highmem-8
246: n1-standard-16
247: n1-highmem-16
Project Member

Comment 4 by bugdroid1@chromium.org, Jun 15 2018

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infra/infra_internal/+/fea6476ec81b3486d562b18b343c2969b38b88a1

commit fea6476ec81b3486d562b18b343c2969b38b88a1
Author: Elliott Friedman <friedman@google.com>
Date: Fri Jun 15 01:57:59 2018

Comment 5 by fried...@google.com, Jun 15 2018

They're up, but I'm going to leave this open until we finalize it.
Project Member

Comment 6 by bugdroid1@chromium.org, Jun 15 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build/+/5903a1e038f51f827cbfcc2ce99c3c7d5ed83faf

commit 5903a1e038f51f827cbfcc2ce99c3c7d5ed83faf
Author: Peter Collingbourne <pcc@google.com>
Date: Fri Jun 15 20:48:22 2018

Assign slave{244..247}-c1 to "Android CFI Builder Perf FYI".

Each of these machines has a different configuration:
244: n1-standard-8
245: n1-highmem-8
246: n1-standard-16
247: n1-highmem-16

This is part of an experiment to discover the minimum capacity
required to run Android ThinLTO perf bots. Once we have found the
minimum, this change will be reverted and we will update the
configuration of slave270-c1.

Bug: 821860,  853050 
Change-Id: I99ad99c396a0cad6c731a7972dc22bef5d0b80aa
Reviewed-on: https://chromium-review.googlesource.com/1102178
Commit-Queue: Peter Collingbourne <pcc@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>

[modify] https://crrev.com/5903a1e038f51f827cbfcc2ce99c3c7d5ed83faf/masters/master.chromium.perf.fyi/slaves.cfg

Project Member

Comment 7 by bugdroid1@chromium.org, Jun 15 2018

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infradata/master-manager/+/7e704c838fe8bfea2d9ae164b4a4b78c5d303d10

commit 7e704c838fe8bfea2d9ae164b4a4b78c5d303d10
Author: Shenghua Zhang <shenghuazhang@google.com>
Date: Fri Jun 15 21:36:35 2018

How's the testing going?
Summary: Request downgrade of slave270-c1 and slave339-c1 to n1-standard-8 (was: Request downgrade of slave270-c1 and slave339-c1 to n1-highmem-8)
I updated issue 821860 with my results but I forgot to follow up on this bug, sorry about that.

As I wrote there n1-standard-8 seems to be sufficient for this bot and I'd like to request a downgrade to that config.
Project Member

Comment 10 by bugdroid1@chromium.org, Jul 13

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/infra/infra_internal/+/7c48901a94336593db9f7de457829f89c915fffe

commit 7c48901a94336593db9f7de457829f89c915fffe
Author: Elliott Friedman <friedman@google.com>
Date: Fri Jul 13 18:28:16 2018

Status: Fixed (was: Assigned)
Project Member

Comment 12 by bugdroid1@chromium.org, Jul 20

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build/+/c38d4ba2777a93d09121183258d229cfadd962fa

commit c38d4ba2777a93d09121183258d229cfadd962fa
Author: Peter Collingbourne <pcc@chromium.org>
Date: Fri Jul 20 19:24:03 2018

Revert "Assign slave{244..247}-c1 to "Android CFI Builder Perf FYI"."

This reverts commit 5903a1e038f51f827cbfcc2ce99c3c7d5ed83faf.

Reason for revert:
The experiment has finished and we have downgraded slave270-c1
to n1-standard-8.

Original change's description:
> Assign slave{244..247}-c1 to "Android CFI Builder Perf FYI".
> 
> Each of these machines has a different configuration:
> 244: n1-standard-8
> 245: n1-highmem-8
> 246: n1-standard-16
> 247: n1-highmem-16
> 
> This is part of an experiment to discover the minimum capacity
> required to run Android ThinLTO perf bots. Once we have found the
> minimum, this change will be reverted and we will update the
> configuration of slave270-c1.
> 
> Bug: 821860,  853050 
> Change-Id: I99ad99c396a0cad6c731a7972dc22bef5d0b80aa
> Reviewed-on: https://chromium-review.googlesource.com/1102178
> Commit-Queue: Peter Collingbourne <pcc@chromium.org>
> Reviewed-by: Dirk Pranke <dpranke@chromium.org>
> Reviewed-by: John Budorick <jbudorick@chromium.org>

TBR=dpranke@chromium.org,pcc@chromium.org,jbudorick@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 821860,  853050 
Change-Id: I5fa0180fc70b0f673abea6d58b0024d219d28580
Reviewed-on: https://chromium-review.googlesource.com/1142903
Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Peter Collingbourne <pcc@chromium.org>

[modify] https://crrev.com/c38d4ba2777a93d09121183258d229cfadd962fa/masters/master.chromium.perf.fyi/slaves.cfg

Sign in to add a comment