samldb and associated patches

[Kamen Mazdrashki]

Hi Matthias,

On Fri, Sep 24, 2010 at 08:59, Matthias Dieter Wallnöfer <mdw at samba.org>wrote:

> Hi Kamen,
>
> Kamen Mazdrashki wrote:
>
>>
>> http://gitweb.samba.org/?p=mdw/samba.git;a=commitdiff;h=b439112915af7b6b630c8010bfc7fdcbc5b82ac8
>> I think we should leave Python tests with just "show_deleted:1" control.
>> We are deleting a normal objects and thus the result should be either
>> Tombstone
>> or Deleted-object, but no Recycled object.
>>
> I will drop this.
>
> Please, leave "dsscync" part of this commit, it should be ok.


>
>>
>> http://gitweb.samba.org/?p=mdw/samba.git;a=commitdiff;h=4910268373b699b28691599bd56bd2c0cf2b8db0
>> The test checks only repl_meta_data.c behavior, not subtree_delete.
>> Do we have a test for subtree_delete module? I can't find one.
>> I think it is ok to go 'in' this way, but it is good for us to have
>> a subtree_delete tests.
>>
> Isn't "test_delete" under ldap.py enough?
>
> Yeah. It tests if repl_meta_data handles delete of leaf objects correctly.
I worried about interaction between subtree_delete and repl_meta_data
in case we delete not a leaf object - for instance Server object
under Servers DN. In this case we have a mixture of objects with
DISALLOW_MOVE_ON_DELETE flag set and objects w/o this
flag. Thus, just few of those objects are moved in Deleted Objects
container.
Do you have time to write such a test?
If not, I can do it next wee presumably and ask you for review


>
>>    I've also not pushed the timeout change for the reasons Kamen
>>    indicated.
>>
>>
>> http://gitweb.samba.org/?p=mdw/samba.git;a=commitdiff;h=987585e0a6fd2165841bf03eea9675fb5d7c9608
>> This commit really solves the problem temporary.
>> If you happen to browse a heavy javascript-driven by the time
>> you are running 'make test', you may hit this problem again :)
>> We either should implement this wait with several retries
>> or implement role transfer operation to be synchronous from
>> client-side point of view.
>>
>> Dropped
>
> Thanks. I will try to come up with something to solve the
problem in more generic way.

>

-- 
CU,
Kamen