Bug 193734 - webkitpy: Wait longer when launching WebKitTestRunner
Summary: webkitpy: Wait longer when launching WebKitTestRunner
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jonathan Bedard
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2019-01-23 12:44 PST by Jonathan Bedard
Modified: 2019-01-24 08:37 PST (History)
7 users (show)

See Also:


Attachments
Patch (1.75 KB, patch)
2019-01-23 12:49 PST, Jonathan Bedard
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Bedard 2019-01-23 12:44:31 PST
Originally, I only saw this problem when booting simulators. Recently, I saw this timeout hit well after the simulators were booted. The wait time was a bit arbitrary in the first place, I'd like to increase it.
Comment 1 Jonathan Bedard 2019-01-23 12:49:33 PST
Created attachment 359924 [details]
Patch
Comment 2 WebKit Commit Bot 2019-01-23 13:28:41 PST
Comment on attachment 359924 [details]
Patch

Clearing flags on attachment: 359924

Committed r240356: <https://trac.webkit.org/changeset/240356>
Comment 3 WebKit Commit Bot 2019-01-23 13:28:42 PST
All reviewed patches have been landed.  Closing bug.
Comment 4 Radar WebKit Bug Importer 2019-01-23 13:29:29 PST
<rdar://problem/47491778>
Comment 5 Daniel Bates 2019-01-23 17:41:22 PST
Comment on attachment 359924 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=359924&action=review

> Tools/Scripts/webkitpy/port/simulator_process.py:98
> +        with Timeout(15, RuntimeError('Timed out waiting for pid {} to connect at port {}'.format(self._pid, self._target_host.listening_port()))):

I did a double take when I saw this change. This timeout value seems to be following Moore's law. Check it out:

3 seconds @ <http://trac.webkit.org/r211370> (01/30/2017)
6 seconds @ <http://trac.webkit.org/r214896 > (04/04/2017)
*NOW* 15 seconds  (01/23/2019)

We are wasting engineering hours progressively increasing this value. I think we should just save us 10 years and pick ~3072 seconds :P. All joking aside, too bad we cannot do anything better.
Comment 6 Jonathan Bedard 2019-01-24 08:37:05 PST
(In reply to Daniel Bates from comment #5)
> Comment on attachment 359924 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=359924&action=review
> 
> > Tools/Scripts/webkitpy/port/simulator_process.py:98
> > +        with Timeout(15, RuntimeError('Timed out waiting for pid {} to connect at port {}'.format(self._pid, self._target_host.listening_port()))):
> 
> I did a double take when I saw this change. This timeout value seems to be
> following Moore's law. Check it out:
> 
> 3 seconds @ <http://trac.webkit.org/r211370> (01/30/2017)
> 6 seconds @ <http://trac.webkit.org/r214896 > (04/04/2017)
> *NOW* 15 seconds  (01/23/2019)
> 
> We are wasting engineering hours progressively increasing this value. I
> think we should just save us 10 years and pick ~3072 seconds :P. All joking
> aside, too bad we cannot do anything better.

I wish I had a better answer. This is closely related to problems booting simulators, basically, simulators are quite slow. We were getting into cases where bots where 30 minutes into a test run and hit this timeout because the machine was stuck gathering crashlogs from another test run while trying to start up the test runner.