You are viewing a plain text version of this content. The canonical link for it is here.
Posted to test-dev@httpd.apache.org by Doug MacEachern <do...@covalent.net> on 2001/12/04 19:16:16 UTC
Re: [patch] adding the startup polling functionality
On Fri, 23 Nov 2001, Stas Bekman wrote:
> I've figured out that I have to run many times t/TEST -run
> until it successfully pings the server under t/TEST -d, so I want the
> polling functionality for -run too. Since now we have 3 places where the
> polling happens (start/ping/run), this patch:
i like -run the way it is because it doesn't actually send any requests to
the server, just uses ->ping. i've been using this for profiling for
example: t/TEST -times 10 -run t/ssl/verify 2
i don't want the /index.html request included in the profile data.
i also often do:
t/TEST -d lwp -run > test.log
and don't want extra "waiting for server ..." output in the log.
and would rather not see a new -poll option.
can you make t/TEST -ping=block -run ... work for the functionality
you're after?
Re: [patch] adding the startup polling functionality
Posted by Doug MacEachern <do...@covalent.net>.
On Wed, 5 Dec 2001, Stas Bekman wrote:
> with mnemonic: block until ping and then run? the problem is that ping
> is an exit option, will have to change things a bit.
minor change, see recent commit.
> I still like the new -poll option, since in the future we may extend
> other options functionalities to poll too.
let's wait to see if we really need it.
Re: [patch] adding the startup polling functionality
Posted by Stas Bekman <st...@stason.org>.
Doug MacEachern wrote:
> On Wed, 5 Dec 2001, Stas Bekman wrote:
>
>
>>I didn't suggest to change how the default -run works (see the patch).
>>
>
> but you did ask:
> "- any reason for not making -poll turned on by default for -run?"
sorry, I've missed this one.
>>I've suggested a new option -poll, which when used will poll the server
>>till it starts. since it's the same as -ping=block for ping, I've
>>removed the =block part.
>
> right. i suggested making '-ping=block -run' work as expected rather than
> adding a new -poll option.
with mnemonic: block until ping and then run? the problem is that ping
is an exit option, will have to change things a bit.
I still like the new -poll option, since in the future we may extend
other options functionalities to poll too.
_____________________________________________________________________
Stas Bekman JAm_pH -- Just Another mod_perl Hacker
http://stason.org/ mod_perl Guide http://perl.apache.org/guide
mailto:stas@stason.org http://ticketmaster.com http://apacheweek.com
http://singlesheaven.com http://perl.apache.org http://perlmonth.com/
Re: [patch] adding the startup polling functionality
Posted by Doug MacEachern <do...@covalent.net>.
On Wed, 5 Dec 2001, Stas Bekman wrote:
> I didn't suggest to change how the default -run works (see the patch).
but you did ask:
"- any reason for not making -poll turned on by default for -run?"
> I've suggested a new option -poll, which when used will poll the server
> till it starts. since it's the same as -ping=block for ping, I've
> removed the =block part.
right. i suggested making '-ping=block -run' work as expected rather than
adding a new -poll option.
Re: [patch] adding the startup polling functionality
Posted by Stas Bekman <st...@stason.org>.
Doug MacEachern wrote:
> On Fri, 23 Nov 2001, Stas Bekman wrote:
>
>
>>I've figured out that I have to run many times t/TEST -run
>>until it successfully pings the server under t/TEST -d, so I want the
>>polling functionality for -run too. Since now we have 3 places where the
>>polling happens (start/ping/run), this patch:
>>
>
> i like -run the way it is because it doesn't actually send any requests to
> the server, just uses ->ping. i've been using this for profiling for
> example: t/TEST -times 10 -run t/ssl/verify 2
>
> i don't want the /index.html request included in the profile data.
>
> i also often do:
>
> t/TEST -d lwp -run > test.log
>
> and don't want extra "waiting for server ..." output in the log.
>
> and would rather not see a new -poll option.
> can you make t/TEST -ping=block -run ... work for the functionality
> you're after?
I didn't suggest to change how the default -run works (see the patch).
I've suggested a new option -poll, which when used will poll the server
till it starts. since it's the same as -ping=block for ping, I've
removed the =block part.
_____________________________________________________________________
Stas Bekman JAm_pH -- Just Another mod_perl Hacker
http://stason.org/ mod_perl Guide http://perl.apache.org/guide
mailto:stas@stason.org http://ticketmaster.com http://apacheweek.com
http://singlesheaven.com http://perl.apache.org http://perlmonth.com/