1218010 - Shorten the poll interval waiting for httpd.js startup

Status: RESOLVED FIXED

(Testing :: Mochitest, defect)

Description

[Chris Manchester (limited bugmail, email directly)]

We wait for the presence of "server_alive.txt" before starting the test. We poll every second for it -- I noticed in the automation logs we always wait about a second. If I drop the polling interval down to 100ms locally it starts up on the second iteration.

A small thing of course, but it's perceptible locally (and will have an effect for every dir).

Comment 1

[Chris Manchester (limited bugmail, email directly)]

Attachment: MozReview Request: Bug 1218010 - Shorten the polling interval when waiting for httpd.js startup in mochitest. r=ahal

Bug 1218010 - Shorten the polling interval when waiting for httpd.js startup in mochitest. r=ahal

I noticed in automation logs that we always wait about 1s for httpd.js to be
ready -- this is the minimum we could wait if we wait at all, because we poll
every second. Bringing the polling interval down to 100ms locally shows we
succeed after waiting just once. This is a perceptible speedup, and will
have an effect for every dir now that we run-by-dir.

Comment 2

[Andrew Halberstadt [:ahal]]

Comment on attachment 8678360 [details]
MozReview Request: Bug 1218010 - Shorten the polling interval when waiting for httpd.js startup in mochitest. r=ahal

https://reviewboard.mozilla.org/r/23189/#review20669

Good catch! Any reason not to bump it down even further to .05 or even .01?

Comment 3

[Chris Manchester (limited bugmail, email directly)]

(In reply to Andrew Halberstadt [:ahal] from comment #2)
> Comment on attachment 8678360 [details]
> MozReview Request: Bug 1218010 - Shorten the polling interval when waiting
> for httpd.js startup in mochitest. r=ahal
> 
> https://reviewboard.mozilla.org/r/23189/#review20669
> 
> Good catch! Any reason not to bump it down even further to .05 or even .01?

Don't think so. Let's go with .05.

Comment 4

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/9c97989cf231

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/9c97989cf231