[SSSD] [PATCH] CIFS idmap Plugin using SSSD

Benjamin Franzke benjaminfranzke at googlemail.com
Wed Oct 9 13:50:11 UTC 2013


Hi Sumit,

Sorry for the delay, got sidetracked by trying to find an appropriate way
to handle the well known sids etc..
But thanks to your reminder I got over to do the integration into sssd's
autoconf/make now :)
So here is the patch as an attachment.
(as recommended by the contrib page - but if you like it better for review
l'll git send-email it as well..)

This patch does not include an update for the rpm specs.
Could you do that integration? For me as a gentoo user that is currently an
unknown-area :)

Regards, Ben

2013/10/9 Sumit Bose <sbose at redhat.com>

> On Mon, Oct 07, 2013 at 12:22:13PM +0200, Benjamin Franzke wrote:
> > 2013/10/7 Sumit Bose <sbose at redhat.com>
> >
> > > On Thu, Oct 03, 2013 at 11:57:07PM +0200, Benjamin Franzke wrote:
> > > > >
> > > > > Hi Ben,
> > > > >
> > > > > this is really great, thank you very much for the contribution! As
> a
> > > > > matter of fact, we've been planning on delivering the CIFS plugin
> as
> > > > > part of the next release:
> > > > > https://fedorahosted.org/sssd/ticket/1534
> > > > >
> > > > > Have you found the design page we created?
> > > > >
> > >
> https://fedorahosted.org/sssd/wiki/DesignDocs/IntegrateSSSDWithCIFSClient
> > > >
> > > >
> > > > Yeah, I've read those pages before writing the plugin..
> > > >
> > > >
> > > > >
> > > > > Mostly the work on our side was done by Sumit Bose (I've CC-ed
> him). I
> > > > > would propose to get in touch with him to avoid duplicating some
> > > effort.
> > > > >
> > > > Unfortunately Sumit is in Germany and there is a holiday so he won't
> be
> > > > > available until Monday.
> > > > >
> > > >
> > > > > In general, I would suggest to merge this code into the SSSD tree
> if
> > > you
> > > > > wouldn't mind. I think it would makes sense to have the code in the
> > > same
> > > > > repo as the ID mapping library.
> > > > >
> > > >
> > > > I'd really appreciate having it integrated in sssd tree,
> > > > to have centralized maintainance and adoption by distributions..
> > >
> > > Thanks a lot for the patch. I would suggest to put your code into
> > > src/lib/cifs_idmap_sss/ and modify SSSD's Makefile.am to build the
> > > plugin. Would you like to do the integration and send the result for
> > > review to this list again?
> > >
> >
> > I'd like to do that integration, will send a patch..
>
> I'd like to add the plugin to the next Fedora release which
> unfortunately has its deadline next week. I don't want to rush you, but
> do you have a chance to send the patch today or tomorrow? Otherwise
> would you mind if I prepare the patch to integrate your code into the
> SSSD tree?
>
> Thanks for your help.
>
> bye,
> Sumit
> >
> >
> > > About the well-know SIDs. They should not be handled inside the plugin
> > > but by SSSD itself. If you are interested in adding them I'd be happy
> to
> > > help you how to start (if my help is needed at all :-).
> > >
> >
> > Yup, sounds better to handle the SIDs by SSSD core. I'd like to try..
> > I'll see whether time permits me to do that soon.. (doing this as a part
> of
> > my thesis)
> >
> >
> >
> > >
> > > bye,
> > > Sumit
> > >
> > > >
> > > >
> > > > > Thanks again for the contribution, much appreciated!
> > > > > _______________________________________________
> > > > > sssd-devel mailing list
> > > > > sssd-devel at lists.fedorahosted.org
> > > > > https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
> > > > >
> > >
> > > > _______________________________________________
> > > > sssd-devel mailing list
> > > > sssd-devel at lists.fedorahosted.org
> > > > https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
> > >
> > > _______________________________________________
> > > sssd-devel mailing list
> > > sssd-devel at lists.fedorahosted.org
> > > https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
> > >
>
> > _______________________________________________
> > sssd-devel mailing list
> > sssd-devel at lists.fedorahosted.org
> > https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20131009/b9919af0/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-CIFS-idmap-plugin.patch
Type: application/octet-stream
Size: 12988 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20131009/b9919af0/attachment.obj>


More information about the sssd-devel mailing list