On 05/30/2016 05:49 PM, Petr Cech wrote:
On 05/30/2016 10:42 AM, Pavel Březina wrote:
> On 05/27/2016 04:32 PM, Petr Cech wrote:
>> I have new version of this patch set.
>> I fixed CI tests on debian . My thanks belongs to Lukas and Nikolai.
>>  http://sssd-ci.duckdns.org/logs/job/44/04/summary.html
> Ack to the patches, I'm running CI now. The only thing I'm worried about
> is that we're using blocking calls getpwnam_r and similar. We already do
> this on some parts of sssd but looking into the cases it's either
> initialization or a very rare condition or a data provider.
> Here we block for every local object and since this is an NSS responder
> I think we should create a non-blocking way rather sooner than later. It
> will be for free when sssd will manage local users though so I'm not
> sure if it's worth the work.
> sssd-devel mailing list
thanks for review.
I know that CI tests failed .
Actually this CI tests passed .
With your patches, two tests sometimes fails and sometimes not. With
master it seems to always succeed.
I do not know whether it is related to your patches or it is a random bug.