On Fri, Apr 22, 2016 at 05:34:47PM +0200, Sumit Bose wrote: > On Fri, Apr 22, 2016 at 05:17:29PM +0200, Jakub Hrozek wrote: > > On Fri, Apr 22, 2016 at 05:03:06PM +0200, Lukas Slebodnik wrote: > > > On (22/04/16 15:41), Jakub Hrozek wrote: > > > >On Tue, Apr 19, 2016 at 04:11:54PM +0200, Sumit Bose wrote: > > > >> Hi, > > > >> > > > >> this is a follow-up patch to 5a5f1e1053415efaa99bb4d5bc7ce7ac0a95b757 > > > >> which removes another call to ipa_check_master() which might cause an > > > >> infinite loop on an IPA client if the server does not support views. > > > >> > > > >> Please note that this patch is a different one than the one I used for > > > >> the first test build in > > > >> https://bugzilla.redhat.com/show_bug.cgi?id=1328108. See comment #12 > > > >> for > > > >> details. > > > >> > > > >> bye, > > > >> Sumit > > > > > > > >I tested the patch with an SSSD client enrolled against an IPA 3.x > > > >server without views. Lookups and authentication worked fine. > > > > > > > >So ACK from me, but it would be nice to wait for a couple of more days > > > >for confirmation from the original reporter.. > > > Timo confirmed this patch on IRC few day(s) ago. > > > Is it enough? > > > > But IIRC it was a first version of the patch, then Sumit changed it.. > > no, Timo used the new version, the reporter from rhbz#1328108 tested the > old version. > > bye, > Sumit
Ah, OK. Then ACK. _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/sssd-devel@lists.fedorahosted.org