CQ test run came back green even though tests failed |
||||
Issue descriptionIn https://build.chromium.org/p/tryserver.chromium.android/builders/linux_android_rel_ng/builds/318560 net_unittests failed and was marked red, but the overall test run was green despite its status being "Build successful failed net_unittests (with patch) on android" This resulted in the CL getting through the CQ and landing, where it broke net_unittests on the waterfall. What happened here? :/
,
Jun 16 2017
,
Jun 19 2017
,
Jun 21 2017
Just happened to me during a dry run: https://codereview.chromium.org/2948113002/#ps1
,
Jun 21 2017
Also happened to me on a different CL: https://build.chromium.org/p/tryserver.chromium.android/builders/android_n5x_swarming_rel/builds/204942
,
Jul 14 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build/+/c4536ad939c07a96ca8d71272293d9231f36940e commit c4536ad939c07a96ca8d71272293d9231f36940e Author: John Budorick <jbudorick@chromium.org> Date: Fri Jul 14 21:53:47 2017 [api.test_utils] Retain errors from run_tests in run_tests_with_patch. Bug: 734084 Change-Id: I19d5093f12a4d610f48972022d4950f91cf5582b Reviewed-on: https://chromium-review.googlesource.com/546717 Commit-Queue: John Budorick <jbudorick@chromium.org> Reviewed-by: Michael Case <mikecase@chromium.org> Reviewed-by: Benjamin Pastene <bpastene@chromium.org> [modify] https://crrev.com/c4536ad939c07a96ca8d71272293d9231f36940e/scripts/slave/README.recipes.md [modify] https://crrev.com/c4536ad939c07a96ca8d71272293d9231f36940e/scripts/slave/recipe_modules/test_utils/api.py [add] https://crrev.com/c4536ad939c07a96ca8d71272293d9231f36940e/scripts/slave/recipe_modules/chromium_tests/tests/api/run_tests_on_tryserver.expected/nonzero_exit_code_no_gtest_output.json [modify] https://crrev.com/c4536ad939c07a96ca8d71272293d9231f36940e/scripts/slave/recipe_modules/chromium_tests/tests/api/run_tests_on_tryserver.py
,
Jul 14 2017
,
Jul 26 2017
Issue 736176 has been merged into this issue. |
||||
►
Sign in to add a comment |
||||
Comment 1 by agrieve@chromium.org
, Jun 16 2017