[PATCH] [SAMBA4] Add a partitions module to LDB

[simo]

On Fri, 2006-06-09 at 09:08 -0700, Andrew Bartlett wrote:
> On Fri, 2006-06-09 at 10:30 -0400, simo wrote:
> > On Thu, 2006-06-08 at 17:27 -0700, Andrew Bartlett wrote:
> > 
> > > I am already finding bugs with this module.  I'll need to do some very
> > > extensive testing (and perhaps write additional testing scripts).
> 
> To perhaps make this clearer:  I'm already finding bugs in Samba4
> generally, which show up under this module, because sub searches from ""
> as a basedn no longer work.

you should probably search only in the main partition when "" basedn is
provided.

> > > In any case, I've attached my current patch.  I want to commit it at
> > > some point soon, disabled until we are happy with the testing.
> > > 
> > > Comments?
> > 
> > I'd like you to commit that code when you will have some test ready that
> > will show it works.
> 
> I'll work to make 'make test' pass, as well as some specific tests for
> the rename path (I'll try to rename a user into cn=configuration).
> 
> Naturally, the primary purpose is to direct partitions across into a
> remote LDAP backend, so I'll be testing that too.

This will be tricky, but it is necessary that we test it.

Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer
email: idra at samba.org
http://samba.org