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.
* master: 57d8b4b9254442a568838cb60ea16068965f2df0
* sssd-1-13: e5fbaf42f87725e42a40e542c06d5f4cdf4c52c2