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

Issue 611222 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

Create a standardized manual servo V3 update test

Reported by jrbarnette@chromium.org, May 11 2016

Issue description

Prior to pushing a new servo V3 build to the lab, we run through
a testing procedure describe here:
    https://sites.google.com/a/google.com/chromeos/for-team-members/infrastructure/chromeos-admin/update-beaglebone-servo

That procedure has two notable problems:
 1) The procedure requires a local servo V3 on your desktop.
 2) The procedure tests servod sanity, but doesn't check AU
    sanity.

To date, the procedure has been adequate to the task, but we
should do better.  The lab tools already have a script called
update_beaglebone.  We should augment or replace the documented
test procedure with the following steps:
  * Select a DUT with a servo from the lab
  * Lock the DUT.
  * Run update_beaglebone to get to the target version.
  * Run a servo sanity check of some sort (maybe just servo-stat).
  * Run update_beaglebone to revert the target servo V3.
  * Unlock the target DUT.

Better still, we could write a script to perform most of the steps
automatically (DUT selection is likely easier as a manual procedure).

 

Comment 1 by autumn@chromium.org, May 16 2016

Labels: Hotlist-Fixit
Along these lines, I had a made a feature request for automated testing of servo v3 changes using a "staging update pool". See  bug 594204 .
Owner: jrbarnette@chromium.org
Status: Fixed (was: Available)
The documentation URL in the description now has a new procedure
that includes AU testing.  The change happened some time around
mid-August 2016.

So, now fixed.

Labels: cros-infra-fixedit-q117

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

Labels: VerifyIn-59

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

Labels: VerifyIn-60

Comment 7 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

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

Status: Archived (was: Fixed)

Sign in to add a comment