GitLab

This reverts commit bcb0488a.

Reason for revert: Bug fixed

Original change's description:
> Revert "Reland "Only retry failed tests for layout tests""
> 
> This reverts commit c24c2363.
> 
> 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 23ea7f24.
> > 
> > Reason for revert: Issue fixed
> > 
> > Original change's description:
> > > Revert "Only retry failed tests for layout tests"
> > > 
> > > This reverts commit ecb97ada.
> > > 
> > > 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-Commit-Position: refs/heads/master@{#552484}

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: I87279f430e6bee58068bad4f3bce71d1923e0834
Reviewed-on: https://chromium-review.googlesource.com/1024719Reviewed-by: Stephen Martinis <martiniss@chromium.org>
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552968}