[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: (ITS#5470) Sporadic failures with RE24



luca@OpenLDAP.org wrote:
> luca@OpenLDAP.org wrote:
>> This is a multi-part message in MIME format.
>> --------------080809000906010300090306
>> Content-Type: text/plain; charset=ISO-8859-1; format=flowed
>> Content-Transfer-Encoding: 7bit
>>
>> Howard Chu wrote:
>>
>>> Thanks. Please try HEAD again.
>>>
>> No way.
>> new testrun directory in
>> ftp://ftp.sys-net.it/luca_scamoni_its5470_20080430-new.tgz
>>
>> backtrace attached
>>
> recent commits seem to have fixed it (at least, right now I'm not able
> to reproduce it anymore...)

Right. Confirmed here too; I (temporarily) added an assert(0) to the offending 
branch of code to make sure the patch was actually getting hit. It takes a 
very particular timing to trigger that code path.

I'm not sure how we can reliably test for this down the road. Perhaps we 
should add a "disabled" config keyword for backends and syncrepl consumers, so 
that we can start up the individual servers, (which takes an unpredictable 
amount of time for each) and then enable various parts in a fixed sequence 
(e.g. 1 second sleeps between ldapmodify/enable requests). Even that's hit or 
miss, because our test database is so small it's unlikely that we can hit the 
window of time on demand.
-- 
   -- Howard Chu
   CTO, Symas Corp.           http://www.symas.com
   Director, Highland Sun     http://highlandsun.com/hyc/
   Chief Architect, OpenLDAP  http://www.openldap.org/project/