[Freeipa-devel] [PATCH] Add ipasam samba passdb backend

[Simo Sorce]

On Mon, 2011-12-05 at 11:42 +0200, Alexander Bokovoy wrote:
> On Wed, 30 Nov 2011, Martin Kosek wrote:
> > On Wed, 2011-11-30 at 10:11 +0100, Sumit Bose wrote:
> > > On Tue, Nov 29, 2011 at 11:25:41PM +0200, Alexander Bokovoy wrote:
> > > > On Tue, 29 Nov 2011, Sumit Bose wrote:
> > > > > @@ -199,10 +216,11 @@ class ADTRUSTInstance(service.Service):
> > > > >                  self.admin_conn.addEntry(entry)
> > > > >  
> > > > >          entry = ipaldap.Entry(self.smb_dom_dn)
> > > > > -        entry.setValues("objectclass", ["sambaDomain", "nsContainer"])
> > > > > +        entry.setValues("objectclass", [self.OBJC_DOMAIN, "nsContainer"])
> > > > >          entry.setValues("cn", self.domain_name)
> > > > > -        entry.setValues("sambaDomainName", self.netbios_name)
> > > > > -        entry.setValues("sambaSID", self.__gen_sid_string())
> > > > > +        entry.setValues(self.ATTR_FLAT_NAME, self.netbios_name)
> > > > > +        entry.setValues(self.ATTR_SID, self.__gen_sid_string())
> > > > > +        entry.setValues(self.ATTR_GUID, str(uuid.uuid4()))
> > > > >          #TODO: which MAY attributes do we want to set ?
> > > > >          self.admin_conn.add_s(entry)
> > > > Could you please also convert this one to .addEntry(entry)?
> > > > I think it is the last one left...
> > > 
> > > This is fixed in freeipa-sbose-0012-3-Fix-some-pylint-warnings.patch.
> > > 
> > > Martin, shall I extract the add_s->addEntry changes into a separate
> > > patch so the they can reviewed indepently of the 6 ipasam patches?
> > > 
> > > bye,
> > > Sumit
> > > 
> > 
> > I think it is OK to review (and push) them in a scope of your patch
> > 0012-3, we don't have to divide them.
> Yes. ACK then.

Pushed all to master.

Simo.

-- 
Simo Sorce * Red Hat, Inc * New York