Bisect hosts do not switch to another device if their test device fail |
|||
Issue descriptionIn 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
,
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
,
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
,
Aug 9 2016
The following revision refers to this bug: https://chrome-internal.googlesource.com/chrome/tools/build_limited/scripts/slave/+/659bbae695ed1e3c29b03f9f28cdd1055b9c4e19 commit 659bbae695ed1e3c29b03f9f28cdd1055b9c4e19 Author: recipe-roller <recipe-roller@chromium.org> Date: Tue Aug 09 23:51:55 2016
,
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
,
Feb 7 2017
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
,
Feb 20 2018
|
|||
►
Sign in to add a comment |
|||
Comment 1 by sheriffbot@chromium.org
, Aug 6 2016