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

Issue 683987 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Last visit > 30 days ago
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug

Blocked on:
issue 684857



Sign in to add a comment

Replace gru-paladin with kevin-paladin

Project Member Reported by bhthompson@google.com, Jan 23 2017

Issue description

The gru-paladin builder lacks HWTests, but at this stage we want HWTests for this platform, so lets swap gru for kevin.

Kevin appears to have a sufficient lab deployment (60 units) for this procedure, if there any concerns please let me know.
 
CLs are up, sequence goes:

Land https://chromium-review.googlesource.com/#/c/431178/
Restart chromeos waterfall
Verify kevin-paladin is green with HWTests at least twice in a row.
Land https://chromium-review.googlesource.com/#/c/431179/
Land https://chromium-review.googlesource.com/#/c/431180/
Restart chromeos waterfall
Land https://chromium-review.googlesource.com/#/c/431181/
Restart chromeos waterfall
Cc: pprabhu@chromium.org
Components: Infra>Client>ChromeOS
pprabhu@ is primary deputy this week, so he owns any necessary
coordination.

From the perspective of the sequencing, there's one other step,
which is someone (the deputy) has to allocate enough DUTs for
the CQ pool.  That will be necessary to get the paladin green.

Status: Started (was: Untriaged)
Please land the experimental CL and assign the bug to me. We can't mark it important in one waterfall restart anyway. I'll get the DUTs, wait for it to go green (or red for its own reasons) and then hand this back to you.
Owner: pprabhu@chromium.org
This CL landed on the wrong bug:

https://chromium-review.googlesource.com/c/431178/
Owner: bhthompson@chromium.org
Let's also land https://chromium-review.googlesource.com/#/c/431180/ before the first (of two) restarts.
If we do land that, it means there will be a period where we lack even build coverage for gru derivatives.

If we are ok with that SGTM.
Owner: pprabhu@chromium.org
oh, yeah. Let's not do that.
We'll do three restarts then..

Filed crbug.com/684857 for restart tonight.
Blockedon: 684857
Kevin paladin is looking green, we just need to allocate the DUTs it seems?

https://uberchromegw.corp.google.com/i/chromeos/builders/kevin-paladin
According to https://sites.google.com/a/google.com/chromeos/for-team-members/infrastructure/chromeos-admin/creating-pools, since we do not have another board for this platform to share the HWTest load, the cq pool requires 24 DUTs.


The suites pool had 30 DUTs, so I've transferred 24 to cq. (leaving the bare minimum 6 for suites).

Final allocation is:
pprabhu@pprabhu:~/chromiumos/src/third_party/autotest/files$ atest host list -b board:kevin | contrib/count_labels -p
      1 av-flexible
      6 bvt
      1 chameleon
      2 chameleon_hdmi
     24 cq
      1 crosperf
     11 cts
      1 groamer
      1 performance
      1 stress3
      6 suites
      1 usb_peripherals
      2 wificasey_8997
      1 wificell
      1 wifichaos


BTW, there is only one working DUT in pool:suites right now (5 are broken and will be fixed by eng-syslab-cros in their routine rotation)
I'm going to turn on HWTest on the CQ to see how this goes.

In the meantime, we should try to:
- Get more DUTs for suites pool, or
- Find another board on this platform to share the HWTest load with (this is the preffered method).
SGTM, thanks!

There should eventually be more systems in there on this platform, but I am not sure of the timelines.

Once this goes green we can make this one important and start turning down gru.
Project Member

Comment 12 by bugdroid1@chromium.org, Jan 27 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/b14ead78159975b6c42fb51fa905b5b4f3dcdc04

commit b14ead78159975b6c42fb51fa905b5b4f3dcdc04
Author: Prathmesh Prabhu <pprabhu@chromium.org>
Date: Thu Jan 26 20:13:40 2017

chromeos_config: Enable HWTests for kevin.

BUG= chromium:683987 
TEST=unittests.

Change-Id: I0453d1e4c81284635c88f77a618996f3d6381648
Reviewed-on: https://chromium-review.googlesource.com/433519
Commit-Ready: Prathmesh Prabhu <pprabhu@chromium.org>
Tested-by: Prathmesh Prabhu <pprabhu@chromium.org>
Reviewed-by: Don Garrett <dgarrett@chromium.org>

[modify] https://crrev.com/b14ead78159975b6c42fb51fa905b5b4f3dcdc04/cbuildbot/config_dump.json
[modify] https://crrev.com/b14ead78159975b6c42fb51fa905b5b4f3dcdc04/cbuildbot/chromeos_config.py

Project Member

Comment 13 by bugdroid1@chromium.org, Feb 4 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/chromite/+/f9827f060f9e119c902e37ad866482fe1f70c0a1

commit f9827f060f9e119c902e37ad866482fe1f70c0a1
Author: Bernie Thompson <bhthompson@google.com>
Date: Sat Feb 04 04:01:51 2017

Remove gru-paladin from the waterfall

BUG= chromium:683987 
TEST=None

Change-Id: Id32ead730512cb77efb67d75cc284598eb595dff
Reviewed-on: https://chromium-review.googlesource.com/431181
Commit-Ready: Bernie Thompson <bhthompson@chromium.org>
Tested-by: Bernie Thompson <bhthompson@chromium.org>
Reviewed-by: Don Garrett <dgarrett@chromium.org>
Reviewed-by: Brian Norris <briannorris@chromium.org>

[modify] https://crrev.com/f9827f060f9e119c902e37ad866482fe1f70c0a1/cbuildbot/config_dump.json
[modify] https://crrev.com/f9827f060f9e119c902e37ad866482fe1f70c0a1/cbuildbot/waterfall_layout_dump.txt
[modify] https://crrev.com/f9827f060f9e119c902e37ad866482fe1f70c0a1/cbuildbot/chromeos_config.py

Owner: bhthompson@chromium.org
Status: Fixed (was: Started)
Everything should now be in place.

Comment 16 by dchan@google.com, Apr 17 2017

Labels: VerifyIn-59

Comment 17 by dchan@google.com, May 30 2017

Labels: VerifyIn-60
Labels: VerifyIn-61

Comment 19 by dchan@chromium.org, Oct 14 2017

Status: Archived (was: Fixed)

Sign in to add a comment