[IPA] SID allocation using DNA plugin

[Andrew Bartlett]

On Fri, 2009-11-06 at 14:27 -0500, Endi Sukma Dewata wrote:
> Andrew,
> 
> ----- "Andrew Bartlett" <abartlet at samba.org> wrote:
> 
> > > I think ideally this problem should be addressed using polymorphism
> > > instead of function pointer. So we need to create 2 subclasses of
> > > the ProvisionBackend called OpenLDAPBackend and FDSBackend. The
> > > base class should include an empty post_setup() and this should be
> > > overriden in the FDSBackend. In the main code the code should
> > > instantiate the class as follows:
> > > 
> > >     if backend_type == "fedora-ds":
> > >         provision_backend = FDSBackend(...)
> > > 
> > >     if backend_type == "openldap":
> > >         provision_backend = OpenLDAPBackend(...)
> > > 
> > >     provision_backend.post_setup()
> > > 
> > > We could also do this on other parts of the code to cleanly separate
> > > FDS and OpenLDAP code. What do you think?
> > 
> > Yes, this is the approach I was trying to head towards.  Just make sure
> > we don't run the LDB specific functions when we are trying to use
> > OpenLDAP or Fedora DS. 
> 
> Please take a look at the attached patches. These patches and the previous
> patch about the relax control are available in this repository:
> 
> http://github.com/endisd/samba/commits/development/
> 
> The ProvisionBackend now has become a base class for LDBBackend,
> ExistingBackend, FDSBackend, and OpenLDAPBackend. All backend-specific
> attributes have been moved into the appropriate classes. Am I in the
> right direction?

Do these pass 'make test' and otherwise work?

(my dsdb-dn branch may fix the OpenLDAP backend)

In any case, the code looks very reasonable, I look forward to merging
it!

Andrew Bartlett

-- 
Andrew Bartlett                                http://samba.org/~abartlet/
Authentication Developer, Samba Team           http://samba.org
Samba Developer, Cisco Inc.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20091107/a411050c/attachment.pgp>