luca(a)OpenLDAP.org wrote:
luca(a)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/