Speed up v8 test runner by making sure slow tests always run first |
|||
Issue descriptionThe V8 status file allows to mark tests as SLOW. Those will be scheduled first to avoid unnecessary prolonging if randomly scheduled in the end. Currently there is no overview over which tests are how slow on which builder. Also, if a manual clean up was done at some time, it happens often later that new slow tests get checked in without being marked slow.
,
Apr 15 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/tools/build.git/+/51f4bf8eb8e8ce73571d189551c1eaa494588e94 commit 51f4bf8eb8e8ce73571d189551c1eaa494588e94 Author: machenbach@chromium.org <machenbach@chromium.org> Date: Fri Apr 15 06:49:44 2016 V8: Highligh slow tests not marked as slow. BUG=601468 Review URL: https://codereview.chromium.org/1885273003 git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/build@299951 0039d316-1c4b-4281-b951-d872f2087c98 [modify] https://crrev.com/51f4bf8eb8e8ce73571d189551c1eaa494588e94/scripts/slave/recipe_modules/v8/api.py [modify] https://crrev.com/51f4bf8eb8e8ce73571d189551c1eaa494588e94/scripts/slave/recipe_modules/v8/test_api.py [add] https://crrev.com/51f4bf8eb8e8ce73571d189551c1eaa494588e94/scripts/slave/recipes/v8.expected/full_tryserver_v8_v8_linux_rel_ng_triggered_slow_tests.json [modify] https://crrev.com/51f4bf8eb8e8ce73571d189551c1eaa494588e94/scripts/slave/recipes/v8.py
,
Apr 27 2016
,
Jun 27 2016
,
Mar 15 2017
The following revision refers to this bug: https://chromium.googlesource.com/v8/v8.git/+/6e376b6c5b573ea755578eda681fb7eda5758cb3 commit 6e376b6c5b573ea755578eda681fb7eda5758cb3 Author: Michael Achenbach <machenbach@chromium.org> Date: Wed Mar 15 12:05:12 2017 [test] Mark slow tests Mark slow tests discovered by manual audit of reported durations. Slow tests will run first, which will lead to higher test runner throughput. Also some tests have flaky timeouts occasionally. BUG=chromium:601468 Change-Id: If9aeb5422e79ff25b867b01c50386a5fdebeb494 Reviewed-on: https://chromium-review.googlesource.com/455736 Reviewed-by: Michael Starzinger <mstarzinger@chromium.org> Commit-Queue: Michael Achenbach <machenbach@chromium.org> Cr-Commit-Position: refs/heads/master@{#43819} [modify] https://crrev.com/6e376b6c5b573ea755578eda681fb7eda5758cb3/test/cctest/cctest.status [modify] https://crrev.com/6e376b6c5b573ea755578eda681fb7eda5758cb3/test/debugger/debugger.status [modify] https://crrev.com/6e376b6c5b573ea755578eda681fb7eda5758cb3/test/mjsunit/mjsunit.status [modify] https://crrev.com/6e376b6c5b573ea755578eda681fb7eda5758cb3/test/test262/test262.status |
|||
►
Sign in to add a comment |
|||
Comment 1 by bugdroid1@chromium.org
, Apr 8 2016