power_FlashVideoSuspend breaks DUTs
Reported by
jrbarnette@chromium.org,
Sep 27 2016
|
||||||||
Issue descriptionFrom time to time I see older model boards (especially ZGB, Mario, and Butterfly) that go offline after running power_FlashVideoSuspend. Those board don't support servo, so if the DUT doesn't wake up, it stays down and has to be repaired manually. The failures drain the BVT pool, and can lead to tests not running. We need to quit running this test on those old boards. I can imagine various things we might do, but the simplest is probably just to make the test depend on the 'servo' label.
,
Sep 29 2016
Do all boards have at least one DUT that connects with servo? I wonder if we'll still run the test on most boards after adding servo label.
,
Sep 29 2016
Not all boards have servo. Those boards won't run this test. Note that I don't envision an acceptable alternative to blocking the test on boards that don't support servo: We can't afford regular manual repairs because of failures like this.
,
Sep 29 2016
A ZGB in the BVT pool went offline last night testing
a Beta release:
http://cautotest/tko/retrieve_logs.cgi?job=/results/78496003-chromeos-test/
This is one of the critically short boards that can't afford
this kind of outage, and can't support servo.
What is our plan to move forward with a CL today?
,
Sep 29 2016
https://chromium-review.googlesource.com/#/c/391212/ I'm moving forward with committing this CL, as soon as the CQ is healthy, and I find a willing reviewer.
,
Oct 1 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromiumos/third_party/autotest/+/ec819e05834cc5ff6806fd4e9ed459e0b63ddf6a commit ec819e05834cc5ff6806fd4e9ed459e0b63ddf6a Author: Richard Barnette <jrbarnette@chromium.org> Date: Thu Sep 29 17:00:15 2016 Restrict power_FlashVideoSuspend to DUTs with servo. The test has been taking DUTs offline. DUTs without servo that fail this test then need manual repair. So, require a servo to run the test. BUG= chromium:650714 TEST=None Change-Id: I70c5908f4a72390404648b9993b68c645b7f59d8 Reviewed-on: https://chromium-review.googlesource.com/391212 Commit-Ready: Richard Barnette <jrbarnette@chromium.org> Tested-by: Richard Barnette <jrbarnette@chromium.org> Reviewed-by: Wu-cheng Li <wuchengli@chromium.org> [modify] https://crrev.com/ec819e05834cc5ff6806fd4e9ed459e0b63ddf6a/client/site_tests/power_FlashVideoSuspend/control.html5
,
Oct 6 2016
We need to stop this test on R54 as well.
,
Oct 6 2016
[Automated comment] Less than 2 weeks to go before stable on M54, manual review required.
,
Oct 7 2016
Hi Richard, Could you take this issue? I think add servo dependencies is enough for fixing the issue.
,
Nov 16 2016
,
Mar 4 2017
,
Apr 17 2017
,
Apr 18 2017
Verified using https://wmatrix.googleplex.com/unfiltered?hide_missing=True&tests=power_FlashVideoSuspend&builds=R59-9460.1.0
,
Jul 10 2017
|
||||||||
►
Sign in to add a comment |
||||||||
Comment 1 by jrbarnette@chromium.org
, Sep 27 2016Owner: owenlin@chromium.org
Status: Assigned (was: Unconfirmed)