[SSSD] [PATCH] Add ldap_autofs_map_master_name option

C. S. csleclish at gmail.com
Sat Jun 1 07:30:47 UTC 2013


Hi Jakub,

Enclosed is the squashed patch.

No worries, re-reading the thread I think I missed a few things early on.
You were suggesting adding an attribute to handle an alias on the master
map, which is why rfc2307_autofs_mobject_map and rfc2307bis_autofs_entry_map
would needed to have been updated(?).

Also, automounter works fine with different master map names, we use it in
production without issue at least. It was my understanding that sssd needs
to know the master map name so that it can expire it from the cache
properly on updates, is that correct? So in sssd.conf you also have
ldap_autofs_map_master_name = auto.slave, right?

Thanks again for your patience, much appreciated. ;-)




On Thu, May 30, 2013 at 9:44 AM, Jakub Hrozek <jhrozek at redhat.com> wrote:

> On Tue, May 28, 2013 at 04:32:00PM -0700, C. S. wrote:
> > (Including once more as an attachment, it looks like something may have
> > wrapped the lines when I had the patch in the body of the message.)
> >
> >
> > On Tue, May 28, 2013 at 4:29 PM, C. S. <csleclish at gmail.com> wrote:
> >
> > > Hi Jakub,
> > >
> > > Thanks for the feedback! Very helpful. Looks like I dropped the LDAP
> map
> > > changes during a merge, that is now included. And I've removed the
> > > strdup(), whoops! ;-)
> > >
> > > cs
> > >
>
> Hi Cove,
>
> this patch is working for me with a couple of additonal changes that I
> attach as a mini-patch. If you like, you can squash it into yours and
> resend, then I think the patch would be good to go.
>
> Basically I think it's enough to have the master map name defined in the
> LDAP options, not the map and entry option lists. Also the option lists
> need the objectclass to be the first entry in the map.
>
> Sorry, I think I confused you with my previous e-mail.
>
> With the attached patch on top of yours, I was able to run automounter
> with a custom master map (MASTER_MAP_NAME="auto.slave" in
> /etc/sysconfig/autofs) just fine.
>
> Thanks again for the contribution.
>
> _______________________________________________
> 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/20130601/40ae1d59/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-ldap_autofs_map_master_name-option.patch
Type: application/octet-stream
Size: 11710 bytes
Desc: not available
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20130601/40ae1d59/attachment.obj>


More information about the sssd-devel mailing list