<div dir="ltr">Hi Jakub,<div><br></div><div><div>Enclosed is the squashed patch.</div></div><div><br></div><div style>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 <span style="color:rgb(0,0,0);white-space:pre-wrap">rfc2307_autofs_mobject_map and </span><span style="color:rgb(0,0,0);white-space:pre-wrap">rfc2307bis_autofs_entry_map would needed to have been updated(?).</span></div>
<div style><br></div><div style>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?</div>
<div style><br></div><div style>Thanks again for your patience, much appreciated. ;-)</div><div style><br></div><div style><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 30, 2013 at 9:44 AM, Jakub Hrozek <span dir="ltr">&lt;<a href="mailto:jhrozek@redhat.com" target="_blank">jhrozek@redhat.com</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Tue, May 28, 2013 at 04:32:00PM -0700, C. S. wrote:<br>
&gt; (Including once more as an attachment, it looks like something may have<br>
&gt; wrapped the lines when I had the patch in the body of the message.)<br>
&gt;<br>
&gt;<br>
&gt; On Tue, May 28, 2013 at 4:29 PM, C. S. &lt;<a href="mailto:csleclish@gmail.com">csleclish@gmail.com</a>&gt; wrote:<br>
&gt;<br>
&gt; &gt; Hi Jakub,<br>
&gt; &gt;<br>
&gt; &gt; Thanks for the feedback! Very helpful. Looks like I dropped the LDAP map<br>
&gt; &gt; changes during a merge, that is now included. And I&#39;ve removed the<br>
&gt; &gt; strdup(), whoops! ;-)<br>
&gt; &gt;<br>
&gt; &gt; cs<br>
&gt; &gt;<br>
<br>
</div>Hi Cove,<br>
<br>
this patch is working for me with a couple of additonal changes that I<br>
attach as a mini-patch. If you like, you can squash it into yours and<br>
resend, then I think the patch would be good to go.<br>
<br>
Basically I think it&#39;s enough to have the master map name defined in the<br>
LDAP options, not the map and entry option lists. Also the option lists<br>
need the objectclass to be the first entry in the map.<br>
<br>
Sorry, I think I confused you with my previous e-mail.<br>
<br>
With the attached patch on top of yours, I was able to run automounter<br>
with a custom master map (MASTER_MAP_NAME=&quot;auto.slave&quot; in<br>
/etc/sysconfig/autofs) just fine.<br>
<br>
Thanks again for the contribution.<br>
<br>_______________________________________________<br>
sssd-devel mailing list<br>
<a href="mailto:sssd-devel@lists.fedorahosted.org">sssd-devel@lists.fedorahosted.org</a><br>
<a href="https://lists.fedorahosted.org/mailman/listinfo/sssd-devel" target="_blank">https://lists.fedorahosted.org/mailman/listinfo/sssd-devel</a><br>
<br></blockquote></div><br></div>