ando(a)OpenLDAP.org wrote:
> Update of /repo/OpenLDAP/pkg/ldap/tests/scripts
>
> Modified Files:
> test049-sync-config 1.3 -> 1.4
>
> Log Message:
> make sure replication finished before comparing data (under valgrind,
> replication may take ages)
I debated adding such a check before. I think the first check you added
should be removed.
That was overparanoid. I added it only for completeness. In fact, the
only case that's creating trouble right now on my laptop is that of the
actual data. Using valgrind with SLAPD_DEBUG=0 it requires two
ldapsearches to actually get the data in. Also, I simply made it check
for the last entry that's added, although there might be no guarantee that
that shows up last in the consumer as well.
In my initial testing there were timing-dependent
errors
that cropped up when refreshing was occurring while the ldapadd was
running.
I think it's important that we continue to test for this case, however
crudely it's done. Too bad we don't have a reliable means to notify the
test
script when the consumer has actually started its work. We could query
back-monitor but normally (not under valgrind) the refresh could complete
before we got the back-monitor search result.
I was also considering the possibility to compare some contextCSN between
producer and consumer. Maybe that's the best way.
p.