New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 730081 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Windows
Pri: 1
Type: Feature



Sign in to add a comment

Please allocate 5 Win7 VMs for optional Win tryserver builders

Project Member Reported by kainino@chromium.org, Jun 6 2017

Issue description

https://luci-milo.appspot.com/buildbot/tryserver.chromium.win/win_optional_gpu_tests_rel/

This bot is backed up and some builds were waiting for at least 90 minutes when I checked.

I haven't been able to identify any particular issue, but some of the recent 'compile (with patch)' times have been very long (~30 minutes)
 

Comment 1 by kbr@chromium.org, Jun 6 2017

Cc: kbr@chromium.org
Components: -Infra Infra>Labs
Labels: -Type-Bug-Regression Type-Feature
Summary: Please allocate 5 Win7 VMs for optional Win tryserver builders (was: win_optional_gpu_tests_rel backed up)
Modifying summary and recategorizing as P1 feature.

Labs team, could you please allocate 5 more win7 VMs for the optional Windows tryserver builders? The pool is defined here:

https://cs.chromium.org/chromium/build/masters/master.tryserver.chromium.win/slaves.cfg?q=master.tryserver.chromium.win/slaves.+package:%5Echromium$&l=1

under "optional_slaves_chromium".

Currently the machines allocated for this are:

vm181-m4
vm185-m4
vm186-m4
vm187-m4
vm192-m4

See https://build.chromium.org/p/tryserver.chromium.win/builders/win_optional_gpu_tests_rel for example (one of the trybots that are backed by this shared pool).

Adding 5 more VMs will double the capacity here.

We could use GCE slaves instead, if those would be better, but in that case we would need to replace these 5 VMs, too.

Thanks very much -- tryjobs are backed up for 4+ hours right now.

Owner: pschmidt@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 3 by bugdroid1@chromium.org, Jun 7 2017

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome-golo/chrome-golo/+/a3192ce00f4dfd812d68b2b8f82d982848ac6554

commit a3192ce00f4dfd812d68b2b8f82d982848ac6554
Author: Peter Schmidt <pschmidt@google.com>
Date: Wed Jun 07 19:44:04 2017

Project Member

Comment 4 by bugdroid1@chromium.org, Jun 7 2017

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

commit 63b7384d189f3cd6b74aa355782991425a1ad5fb
Author: Peter Schmidt <pschmidt@google.com>
Date: Wed Jun 07 22:12:34 2017

Add vm14{72..76}-m4 to optional_slaves_chromium pool on master.tryserver.win

Bug:  730081 
Change-Id: If3d97bad6b67c4bcb223195d156da2362daca78b
Reviewed-on: https://chromium-review.googlesource.com/527812
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Reviewed-by: John Weathersby <johnw@chromium.org>
Reviewed-by: Daniel Jacques <dnj@chromium.org>
Commit-Queue: John Weathersby <johnw@chromium.org>

[modify] https://crrev.com/63b7384d189f3cd6b74aa355782991425a1ad5fb/masters/master.tryserver.chromium.win/slaves.cfg

Comment 5 by kbr@chromium.org, Jun 7 2017

Should I schedule a restart of master.tryserver.chromium.win for the end of the day today?

Sure.

Comment 7 by kbr@chromium.org, Jun 7 2017

Thanks. Scheduled for ~7 PM Pacific today in https://chrome-internal-review.googlesource.com/390790 via:

~/src/depot_tools/cit restart tryserver.chromium.win -m 200 -b 730081


Project Member

Comment 8 by bugdroid1@chromium.org, Jun 7 2017

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

commit 90e54161347fe973ab78ddd8ed46caf9fe1701ee
Author: Peter Schmidt <pschmidt@google.com>
Date: Wed Jun 07 22:43:54 2017

Cleanup unnecessary parenthesis on pool definition in tryserver.chromium.win

Bug:   730081 
Change-Id: I7086d4d927bbeed6c9242c5fff3af5a0a92d39aa
Reviewed-on: https://chromium-review.googlesource.com/527496
Reviewed-by: Daniel Jacques <dnj@chromium.org>
Commit-Queue: Peter Schmidt <pschmidt@chromium.org>

[modify] https://crrev.com/90e54161347fe973ab78ddd8ed46caf9fe1701ee/masters/master.tryserver.chromium.win/slaves.cfg

Project Member

Comment 9 by bugdroid1@chromium.org, Jun 7 2017

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

commit 5fb6f0c23c5670c63aa6df83670f68ab87a7f17e
Author: Kenneth Russell <kbr@google.com>
Date: Wed Jun 07 22:44:54 2017

Comment 10 by kbr@chromium.org, Jun 8 2017

Peter, could you please reboot the 5 new VMs? They're now attached, but they're all marked offline:
https://build.chromium.org/p/tryserver.chromium.win/builders/win_optional_gpu_tests_rel

vm1472-m4       offline	
vm1473-m4       offline	
vm1474-m4       offline	
vm1475-m4       offline	
vm1476-m4       offline

Thanks.

Status: Fixed (was: Assigned)
The new slaves are now connected to the master.

Comment 12 by kbr@chromium.org, Jun 8 2017

Great! Thanks Peter. The queue lengths look a lot better today.

Sign in to add a comment