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

Issue 616541 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Jun 2016
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Add back the PoE-reboot method to servo repair

Project Member Reported by waihong@chromium.org, Jun 1 2016

Issue description

It was removed. Check the code of servo_host.py:

        # TODO(dshi): add self._powercycle_to_repair back to repair_funcs
        # after crbug.com/336606 is fixed.
        repair_funcs = [self._repair_with_sysrq_reboot,]

Should add it back.
 
Cc: waihong@chromium.org
The CL is prepared: https://chromium-review.googlesource.com/#/c/348730
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 4 2016

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

commit 0635dce44bf269f57859fa169ba0b53cc7599538
Author: Tom Wai-Hong Tam <waihong@chromium.org>
Date: Wed Jun 01 18:17:50 2016

[autotest] Add back the power-cycle method to one of the servo repair methods

The PoE support for servos was enabled in the stress lab, should enable it.

BUG= chromium:616541 
TEST=None, trivial

Change-Id: Icc20049ebc23840c26d09b2c21bf50e7512f8269
Reviewed-on: https://chromium-review.googlesource.com/348730
Commit-Ready: Tom Tam <waihong@google.com>
Tested-by: Tom Tam <waihong@google.com>
Reviewed-by: Dan Shi <dshi@google.com>

[modify] https://crrev.com/0635dce44bf269f57859fa169ba0b53cc7599538/server/hosts/servo_host.py

Comment 3 by dshi@chromium.org, Jun 6 2016

Status: Fixed (was: Assigned)
FYI A push to prod test is on going.
Status: Verified (was: Fixed)
Bulk verified

Sign in to add a comment