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
Regards
--
Petr^4 Čech