On 06/17/2016 05:31 PM, Noriko Hosoi
wrote:
On 06/17/2016 12:17 AM, Ludwig
Krispenz wrote:
Hi Noriko,
I still have a doubt on your fix. You now base the entries to be
sent only on the parentid index, but when creating a tombstone,
the entry is removed from the parentid index. So you will miss
the tombstones in the total init.
Well, I don't think a tombstone entry is removed from the parentid
index...
but in ldbm_back_delete() we remove the entryID from all inexes and
if create_tombstone_entry add it back to specific indexes
This is an example of a tombstone in id2entry.db
id 16
rdn:
nsuniqueid=bd90578a-2f6311e6-9346e70f-8f7f52e7,uid=tuser0
entryid: 16
parentid: 20
This is the corresponding item on parentid.db. The entry 16 is
still there as a child of parent id 20...
# dbscan -f parentid.db -k =20 -r -n
=20 3
10 16 19
was dbcache already flushed to disk when you did run dbscan ? I get
dn:
nsuniqueid=bd76ad01-322c11e6-a5389989-c3cc84f8,cn=x,ou=People,dc=example,dc=com
entryid: 30
parentid: 2
dn: cn=y,ou=People,dc=example,dc=com
entryid: 31
parentid: 2
dbscan -f /var/lib/dirsrv/slapd-elkris3/db/example/parentid.db -k
=2 -r -n
=2 1
31
Besides, the tombstone entry is found on the consumer server after
the total update/bulk import...
# ldapsearch [...] -b "dc=example,dc=com"
"(|(objectclass=nstombstone)(objectclass=ldapsubentry))" dn
dn: cn=repl keep alive 10000,dc=example,dc=com
dn:
nsuniqueid=bd90578a-2f6311e6-9346e70f-8f7f52e7,uid=tuser0,ou=ou3,dc=example,dc=com
[...]
But obviously there are still issues, with which it does not pass
the IPA CI test... Continue looking into it...
Thanks,
--noriko
Ludwig
On 06/17/2016 03:05 AM, Noriko
Hosoi wrote:
https://fedorahosted.org/389/ticket/48755
https://fedorahosted.org/389/attachment/ticket/48755/0001-Ticket-48755-moving-an-entry-could-make-the-online-i.3.patch
https://fedorahosted.org/389/attachment/ticket/48755/0002-Ticket-48755-CI-test-test-case-for-ticket-48755.3.patch
git patch file (master) -- additional test case to the CI test
patch
I also built copr build including the patch:
https://copr.fedorainfracloud.org/coprs/nhosoi/389-ds-base-1.3.5.7.f24/builds/
Martin, could it be possible to use this tentative build in
the IPA acceptance test? (I don't want to break IPA again...)
Thanks!
--noriko
--
389-devel mailing list
389-devel@xxxxxxxxxxxxxxxxxxxxxxx
https://lists.fedoraproject.org/admin/lists/389-devel@xxxxxxxxxxxxxxxxxxxxxxx
--
Red Hat GmbH, http://www.de.redhat.com/, Registered seat: Grasbrunn,
Commercial register: Amtsgericht Muenchen, HRB 153243,
Managing Directors: Charles Cachera, Michael Cunningham, Michael O'Neill, Eric Shander
--
389-devel mailing list
389-devel@xxxxxxxxxxxxxxxxxxxxxxx
https://lists.fedoraproject.org/admin/lists/389-devel@xxxxxxxxxxxxxxxxxxxxxxx
--
389-devel mailing list
389-devel@xxxxxxxxxxxxxxxxxxxxxxx
https://lists.fedoraproject.org/admin/lists/389-devel@xxxxxxxxxxxxxxxxxxxxxxx
--
Red Hat GmbH, http://www.de.redhat.com/, Registered seat: Grasbrunn,
Commercial register: Amtsgericht Muenchen, HRB 153243,
Managing Directors: Charles Cachera, Michael Cunningham, Michael O'Neill, Eric Shander
|
