metze at samba.org
Fri Jun 3 09:57:03 UTC 2016
> I've been looking, as part of my work at Catalyst, at issues around
> corruption of our DB due to replication issues. Specifically, rename
> handling, particularly with subtrees.
> I found and fixed a number of bugs, which you will have seen in the
> braches I've proposed. However, I've been less than satisfied with the
> tests for this area, so have been extending them.
> I figured you might be very interested in the latest test findings,
> regarding replPropertyMetaData.
> As far as I can tell, in windows 2012r2, the special sort order for
> replPropertyMetaData, that is with the RDN last, simply doesn't exist.
> Indeed, I wonder if it has always been an artifact of our DRS -> ldb
> conversion code in replicated_objects.c
Could be :-)
I'm also unable to see the rdn attribute metadata in any of LDAP searches
or ndrdumps of replication traces.
I guess the important thing is that we keep the rdn value uptodate/in sync
with the DN and the name attribute.
> I now have tests that query the server for the replPropertyMetaData and
> the DRS equivalent, after each operation, and it shows no such special
> sort order.
> The only special thing I can find is that the RDN (eg CN) is given a
> local invocationID and a local version quite often, while remaining
> excluded from replication (name is replicated instead).
> See http://git.catalyst.net.nz/gw?p=samba.git;a=shortlog;h=refs/heads/r
> Thankfully because this is excluded over DRS, this has not caused
> replication corruption.
> Finally, I do expect that the patches I have already written will cover
> the dbcheck flapping test, which was a genuine DB corruption, due to
> mishandling of renamed/deleted objects.
I don't have time for a deep review, but it looks
promising. Thanks a lot!
One comment regarding
From reading just the diff I'm wondering how this pass against Samba.
I had to add the following trick to get meta data stamps for empty
attributes in order to get
to pass see
This is part of the
branches. I'll write more about that branches later....
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 836 bytes
Desc: OpenPGP digital signature
More information about the samba-technical