new-run-webkit-tests: disable threading on macs by default

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Dirk Pranke]

We are currently trying to diagnose what appear to be several concurrency-related hangs on the Chromium bots. Several bots (Chromium WebKit Mac (webkit.org) canary, Chromium WebKit Mac, Chromium WebKit Mac (dbg)(3) ) seem to be hanging more than 25% of the time.

Until we can diagnose what's going on better, we should consider disabling concurrency by default on the Chromium Mac ports. Most of the bots run two threads in parallel, so this will near-double the cycle time for the layout tests on them.

[Eric Seidel (no email)]

Another possibility would be to try changing DEPS to point to an older version of WebKitTools on those bots and see if the hangs go away.

[Dirk Pranke]

Created attachment 55069 [details]
patch

[Eric Seidel (no email)]

Comment on attachment 55069 [details]
patch

We really should have a unit test for the run_webkit_tests.py change.

This seems OK or now.  Unfortunately it means we'll stop getting crash/hang data, but hopefully we have enough and can repro locally anyway.

Seems we should log to the console when we do this override.

[Eric Seidel (no email)]

Comment on attachment 55069 [details]
patch

Please fix the mentioned nits.  If you aren't going to unit test this we should at least mention why in the ChangeLog.

[Dirk Pranke]

Created attachment 55070 [details]
add log message

[Dirk Pranke]

Created attachment 55075 [details]
add unit tests for the change to run_webkit_tests

[Dirk Pranke]

Committed r58789: <http://trac.webkit.org/changeset/58789>

[Eric Seidel (no email)]

Thanks for the tests!