On 06/19/2015 10:46 AM, Lukas Slebodnik wrote:
On (19/06/15 10:37), Pavel Reichl wrote:
> On 06/18/2015 06:16 PM, Lukas Slebodnik wrote:
>> ehlo,
>>
>> I noticed the same noise in log files as user in ticker 2678.
>> It was a little bit related to the latest changes with setting
>> initgroups flag in the right time.
>>
>> LS
>>
>>
> Hello,
> the patch won't apply on current master. Does it depend on some patch which
> is being reviewed?
Yes,
It is related to "[PATCH] SDAP: Remove user from cache for missing user in
LDAP". I didn't send them together because this patch does not fix regression.
LS
OK thanks, the only nitpick I found is usage of strerror, I know that
Michal has patch to change this globally but I think that we are in
agreement to use sss_strerror in changed/new code anyway, right?
+ } else if (ret != ENOENT) {
DEBUG(SSSDBG_OP_FAILURE,
- "sysdb_search_object_by_uuid did not return
a " \
- "single result.\n");
+ "sysdb_search_object_by_uuid failed or
returned "
+ "more than one result [%d][%s].\n",
+ ret, strerror(ret));
Otherwise code LGTM and CI passed, but I assume that I can't ack before
"[PATCH] SDAP: Remove user from cache for missing user in LDAP" gets
pushed to master anyway.
_______________________________________________
sssd-devel mailing list
sssd-devel(a)lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel