On 08/10/2016 07:30 AM, Petr Cech wrote:
> On 08/09/2016 07:10 PM, Lukas Slebodnik wrote:
> > On (09/08/16 10:21), Jakub Hrozek wrote:
> > > On Mon, Aug 08, 2016 at 06:37:10PM +0200, Lukas Slebodnik wrote:
> > > > ehlo,
> > > >
> > > > yet another patch which fixes issues caused by sysdb refactoring.
> > > >
> > > > reproducer:
> > > > a) add user with two groups
> > > > b) call id user
> > > > c) add another group to user
> > > > d) authenticate
> > > > e) check tha id -G return 3 groups.
> > > >
> > > > I am looking forward to the pam_wrapper So we can test it in
upstream.
> > > >
> > > > LS
> > >
> > > > From e501f791ee2e03b8eaea919a2f8ca4474773e3ba Mon Sep 17 00:00:00
2001
> > > > From: Lukas Slebodnik <lslebodn(a)redhat.com>
> > > > Date: Mon, 8 Aug 2016 17:30:29 +0200
> > > > Subject: [PATCH] NSS: Use correct name for invalidating memory cache
> > > >
> > > > After refactoring of sysdb, we get and internal fully qualified
> > > > name from backend in
org.freedesktop.sssd.dataprovider_rev.initgrCheck
> > > > Previously we got short name and we created fq name in
> > > > nss_update_initgr_memcache. Memory cache still need to use short
names
> > > > if it was specified.
> > >
> > > In fill_pwent() we use the sized_output_name() to store the name in the
> > > memcache, can we use the same here?
> > Sure,
> >
> > LS
>
> Hi Lukas, Jakub,
>
> LGTM, I will wait for new CI.
Hi,
CI passed:
http://sssd-ci.duckdns.org/logs/job/51/22/summary.html
Manual testing passed.
=> ACK