[Freeipa-devel] [PATCH] 324 Add fallback for httpd restarts

[Martin Kosek]

On 10/18/2012 02:47 PM, Rob Crittenden wrote:
> Martin Kosek wrote:
>> Attaching a script I used to reproduce the issue on machine with sysV (RHEL 6.4
>> in my case). With the patch applied, httpd restarts correctly fallback-ed.
>>
>> If you think that the wait is not enough, I can add a more complicated
>> procedure, like this one:
>>
>> wait_time = 5
>> retries = 3
>>
>> for x in xrange(retries):
>>      try:
>>         sleep(wait_time)
>>         http.stop()
>>         sleep(wait_time)
>>         http.start()
>>      except CalledProcessError:
>>         wait_time = wait_time * 2
>>         continue
>>      break
>>
>>
>> ----
>>
>> httpd init script on sysV based platforms cannot guarantee that two
>> consecutive httpd service restarts succeed when run in a small
>> time distance.
>>
>> Add fallback procedure that adds additional waiting time after such
>> failed restart attempt, and then try to stop and start the service
>> again.
>>
>> https://fedorahosted.org/freeipa/ticket/2965
> 
> 
> Should we attempt to retrieve a file to ensure that the service is up? The
> ipa.crt would be a candidate for this.
> 
> rob

Hm, this looks over-engineered from my POV. We already check that ports are
open, right?

Martin