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

Issue 619110 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Last visit > 30 days ago
Closed: Jul 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

DUT repair should try re-installing software if AC power isn't detected

Reported by jrbarnette@chromium.org, Jun 10 2016

Issue description

The Autotest repair process restricts the specific procedures
it tries based on the failure symptoms.  In particular, if a
DUT says "I can't run tests because I'm not plugged in to AC
power", the only repair procedure we try is to turn power on
using RPM.  In particular, we don't try to reinstall the
stable repair image.

ITOT bugs where the DUT has AC power but can't detect it are
a thing, and they're likely enough that they've occurred in
the wild (see  bug 617704 ).  So, when the symptom is "AC power
is not plugged in", repair needs to try re-installing stable
software.

 
Project Member

Comment 1 by bugdroid1@chromium.org, Jun 12 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/52d81dba06490518aa565edd5ffa741d99127b89

commit 52d81dba06490518aa565edd5ffa741d99127b89
Author: Richard Barnette <jrbarnette@chromium.org>
Date: Fri Jun 10 18:25:00 2016

[autotest] In repair, if no power, try reinstalling the software.

It's possible for a DUT to have AC power, yet have a software bug
that causes it to report "no AC power."  So, make "no AC power" a
trigger for all repair actions that reinstall the software.

BUG= chromium:619110 
TEST=None

Change-Id: Ic72e711c32c65cebb9e93d09237837ea51a31d29
Reviewed-on: https://chromium-review.googlesource.com/351501
Commit-Ready: Richard Barnette <jrbarnette@chromium.org>
Commit-Ready: Simran Basi <sbasi@chromium.org>
Tested-by: Richard Barnette <jrbarnette@chromium.org>
Reviewed-by: Kevin Cheng <kevcheng@chromium.org>

[modify] https://crrev.com/52d81dba06490518aa565edd5ffa741d99127b89/server/hosts/cros_repair.py

Status: Fixed (was: Started)
Labels: VerifyIn-54

Comment 4 by dchan@chromium.org, Oct 7 2016

Labels: VerifyIn-55

Comment 5 by dchan@chromium.org, Oct 10 2016

Labels: -VerifyIn-55

Comment 6 by dchan@google.com, Nov 19 2016

Labels: VerifyIn-56

Comment 7 by dchan@google.com, Jan 21 2017

Labels: VerifyIn-57

Comment 8 by dchan@google.com, Mar 4 2017

Labels: VerifyIn-58

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

Labels: VerifyIn-59

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

Labels: VerifyIn-60
Labels: VerifyIn-61

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

Status: Archived (was: Fixed)

Sign in to add a comment