|author||Yuke Liao <email@example.com>||Wed May 15 15:46:39 2019|
|committer||Commit Bot <firstname.lastname@example.org>||Wed May 15 15:46:39 2019|
Revert "[code coverage] Correctly surface failed tests at build level" This reverts commit 4e981183406f446e19115560443c2eb61d07bf7a. Reason for revert: This CL seems to be breaking the coverage CI builders Original change's description: > [code coverage] Correctly surface failed tests at build level > > Test results of the coverage trybot was surfaced at step level, but > not at the build level because the failed tests were ignored in the > recipe, and this CL fixes the issue to correctly surface them. > > Related to the purpose of surfacing test results, this CL also makes > linux-coverage-rel run without patch steps, but ignore the coverage > dumps from those steps. > > Bug: 923711, 944659 > Change-Id: I663062a095f3cc6ff237d3a33c46b9f340f4a0c8 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/build/+/1609579 > Commit-Queue: Yuke Liao <email@example.com> > Reviewed-by: Stephen Martinis <firstname.lastname@example.org> TBRemail@example.com,firstname.lastname@example.org,email@example.com Change-Id: I81af28d7e190fcb3a441487d56afbdaee7d23c9e No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 923711, 944659 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/build/+/1612707 Reviewed-by: Yuke Liao <firstname.lastname@example.org> Commit-Queue: Yuke Liao <email@example.com>
Hi build contributor! If you do any change in
scripts/master/ or touching any master's
html/ directories, you must restart master.chromium.fyi first and ensure that it still works before restarting other masters.
If you're here to make a change to ‘recipes’ (the code located in
scripts/slave/recipes*), please take a look at the README for more information pertaining to recipes.