Revert "Reland "Only retry failed tests for layout tests""
This reverts commit c24c2363652374e2553ddaed2934f07be1624ac6.
Reason for revert: https://chromium-review.googlesource.com/c/chromium/src/+/1018590 broke webkit
Original change's description:
> Reland "Only retry failed tests for layout tests"
>
> This reverts commit 23ea7f24ca948796ee681e50eb61c873a5dcf329.
>
> Reason for revert: Issue fixed
>
> Original change's description:
> > Revert "Only retry failed tests for layout tests"
> >
> > This reverts commit ecb97ada9d911c28d5b85abb036a1fd3b952645c.
> >
> > Reason for revert: https://ci.chromium.org/p/chromium/builders/luci.chromium.try/linux_chromium_rel_ng/73246 looks invalid
> >
> > Original change's description:
> > > Only retry failed tests for layout tests
> > >
> > > Recently layout tests have been fairly flaky; this is causing extra load
> > > on the CQ. Every time layout tests fails, they are retried without patch
> > > to determine if the failure is a flake or a failure at tip of tree. This
> > > means that layout tests are running more than other test suites, and
> > > since layout tests take a long time to run, we end up consuming more
> > > capacity from our swarming pool than we should.
> > >
> > > Isolated script tests don't have the ability to retry only the tests
> > > that failed, which is what other test suites do. This CL changes that so
> > > that layout tests, when re-run without the patch, retry on the tests that
> > > fail, which should significantly reduce swarming load and cycle time for
> > > running layout tests without patch.
> > >
> > > Bug: 533481
> > > Change-Id: I1f23f6e57bcfbc44f80f11a0a96ca74d84ae8423
> > > Reviewed-on: https://chromium-review.googlesource.com/1015843
> > > Commit-Queue: Stephen Martinis <martiniss@chromium.org>
> > > Reviewed-by: Dirk Pranke <dpranke@chromium.org>
> > > Reviewed-by: John Budorick <jbudorick@chromium.org>
> > > Cr-Commit-Position: refs/heads/master@{#551821}
> >
> > TBR=dpranke@chromium.org,martiniss@chromium.org,jbudorick@chromium.org
> >
> > Change-Id: Ia6092ae71aad64689cdac37e6cd6524d087ff428
> > No-Presubmit: true
> > No-Tree-Checks: true
> > No-Try: true
> > Bug: 533481
> > Reviewed-on: https://chromium-review.googlesource.com/1018161
> > Reviewed-by: Stephen Martinis <martiniss@chromium.org>
> > Commit-Queue: Stephen Martinis <martiniss@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#551877}
>
> TBR=dpranke@chromium.org,martiniss@chromium.org,jbudorick@chromium.org
>
> # Not skipping CQ checks because original CL landed > 1 day ago.
>
> Bug: 533481
> Change-Id: If8c572258d9ed3abc020bc5bad56d1b06107ee0f
> Reviewed-on: https://chromium-review.googlesource.com/1022552
> Reviewed-by: Stephen Martinis <martiniss@chromium.org>
> Reviewed-by: Dirk Pranke <dpranke@chromium.org>
> Commit-Queue: Stephen Martinis <martiniss@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#552482}
TBR=dpranke@chromium.org,martiniss@chromium.org,jbudorick@chromium.org
Change-Id: Ibd7819d1adb43895e33b74622438aa61517a036d
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 533481
Reviewed-on: https://chromium-review.googlesource.com/1022558
Reviewed-by: Stephen Martinis <martiniss@chromium.org>
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#552484}
Cr-Mirrored-From: https://chromium.googlesource.com/chromium/src
Cr-Mirrored-Commit: bcb0488a9f5c7bc1e327d144e3982973d9eec6a46 files changed
