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

Issue 635183 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Feb 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Bisect hosts do not switch to another device if their test device fail

Project Member Reported by zxiong@google.com, Aug 5 2016

Issue description

In the example below, when the test device was disconnected from host at step 33, the host did not rerun the tests on its other available device, but continued running using the disconnected device and failed all the tests. 


Example: https://build.chromium.org/p/tryserver.chromium.perf/builders/android_nexus5X_perf_bisect/builds/445
 
Project Member

Comment 1 by sheriffbot@chromium.org, Aug 6 2016

Labels: Hotlist-Google
Project Member

Comment 2 by bugdroid1@chromium.org, Aug 9 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build.git/+/2c78661f30da29ac3d08a6e26b2392ada22dbba4

commit 2c78661f30da29ac3d08a6e26b2392ada22dbba4
Author: zxiong <zxiong@google.com>
Date: Tue Aug 09 23:40:26 2016

Check if device disconnects every time the last test of a revision fails

Right now when the test device is disconnected from host, the host will
not rerun the tests on its other available device, but will continue
using the disconnected device and fail all the tests. This problem can be
solved by checking connected devices in every revision test and raising
step failure if test device disconnects.
BUG= chromium:635183 

Review-Url: https://codereview.chromium.org/2224583003

[modify] https://crrev.com/2c78661f30da29ac3d08a6e26b2392ada22dbba4/scripts/slave/recipe_modules/auto_bisect/revision_state.py

Project Member

Comment 3 by bugdroid1@chromium.org, Aug 9 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build.git/+/2c78661f30da29ac3d08a6e26b2392ada22dbba4

commit 2c78661f30da29ac3d08a6e26b2392ada22dbba4
Author: zxiong <zxiong@google.com>
Date: Tue Aug 09 23:40:26 2016

Check if device disconnects every time the last test of a revision fails

Right now when the test device is disconnected from host, the host will
not rerun the tests on its other available device, but will continue
using the disconnected device and fail all the tests. This problem can be
solved by checking connected devices in every revision test and raising
step failure if test device disconnects.
BUG= chromium:635183 

Review-Url: https://codereview.chromium.org/2224583003

[modify] https://crrev.com/2c78661f30da29ac3d08a6e26b2392ada22dbba4/scripts/slave/recipe_modules/auto_bisect/revision_state.py

Project Member

Comment 4 by bugdroid1@chromium.org, Aug 9 2016

Project Member

Comment 5 by bugdroid1@chromium.org, Aug 10 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/411f3a717cba9dbf92413f6dfa59cd1bb6a6bfc4

commit 411f3a717cba9dbf92413f6dfa59cd1bb6a6bfc4
Author: recipe-roller <recipe-roller@chromium.org>
Date: Wed Aug 10 18:29:51 2016

Roll recipe dependencies (trivial).

This is an automated CL created by the recipe roller. This CL rolls recipe
changes from upstream projects (e.g. depot_tools) into downstream projects
(e.g. tools/build).

More info is at https://goo.gl/zkKdpD. Use https://goo.gl/noib3a to file a bug
(or complain)

build:
  https://crrev.com/1ef1717a1d167fa243e0db9f89b1f715ca90b251 Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/1f32353ff4a7011bbded5898da32981437de16eb [Android] Specify --adb-path when printing perf test results. (jbudorick@chromium.org)
  https://crrev.com/52b73ce928346e5ff2de78658eed76bf048b1d45 Presubmit recipe: rebase Gerrit patches. (tandrii@chromium.org)
  https://crrev.com/ed36f101999bd972cea1b5be8e8e663e4c882a13 [perf] Remove Large Profile Generator. (dtu@chromium.org)
  https://crrev.com/ed494a5ee803bdbb1b0ac7df24cd6d819efc53f5 WebRTC: Disable Linux32 Debug+Release bots for now. (kjellander@chromium.org)
  https://crrev.com/abb024b2505d48e69dd8ba049f7b619677640fe9 Move Flutter engine to src/flutter (abarth@chromium.org)
  https://crrev.com/4e6de3a742791006b6ca41aab6c1bd8868461ce8 Update the path to the travis directory (abarth@chromium.org)
  https://crrev.com/7746bb86402b08a137921c1a2abe07352d95bd74 Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/e6e74c9db3e74f8603acf4d21389439aa20b232c Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
  https://crrev.com/2c78661f30da29ac3d08a6e26b2392ada22dbba4 Check if device disconnects every time the last test of a revision fails (zxiong@google.com)
  https://crrev.com/12b302b6ed29f15838a9145262d7913d19ae4aba Use iossim/test-without-building for Earlgrey tests. (huangml@chromium.org)
  https://crrev.com/a661277f9f72d06f313dba599b4e6a4a7dd9887f Remove the android_isolate_path option from LocalGTestTest. (jbudorick@chromium.org)
depot_tools:
  https://crrev.com/f83fa37b3451a862df10e0a5f8a1265a28e75b1f Add flag to rebase before gclient sync when applying a Gerrit patch (andybons@chromium.org)
  https://crrev.com/1dae95a13f123a16bc1d03bd2c587e770ce2e29f Require internal recipe tryjob (martiniss@chromium.org)
  https://crrev.com/fff0635af2a79778a23be93802f2cabd6c9aedc4 Roll recipe dependencies (trivial). (recipe-roller@chromium.org)
recipe_engine:
  https://crrev.com/87823726a3c8ef0016780ac1b752a758f2191a55 Require internal recipe tryjob (martiniss@chromium.org)

R=dtu@chromium.org,zxiong@google.com,tandrii@chromium.org,huangml@chromium.org,kjellander@chromium.org,jbudorick@chromium.org,andybons@chromium.org,recipe-roller@chromium.org,martiniss@chromium.org,phajdan.jr@chromium.org,ehmaldonado@chromium.org,ianh@google.com,abarth@chromium.org
BUG= 628801 , 633333 ,635975, chromium:635183 , 636080 , 612417 

TBR=martiniss@chromium.org,phajdan.jr@chromium.org

Review-Url: https://codereview.chromium.org/2228243002
Cr-Commit-Position: refs/heads/master@{#411089}

[modify] https://crrev.com/411f3a717cba9dbf92413f6dfa59cd1bb6a6bfc4/infra/config/recipes.cfg

Project Member

Comment 6 by sheriffbot@chromium.org, Feb 7 2017

Labels: Hotlist-Recharge-BouncingOwner
Owner: ----
Status: Untriaged (was: Assigned)
The assigned owner "zxiong@google.com" is not able to receive e-mails, please re-triage.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Status: Fixed (was: Untriaged)

Sign in to add a comment