[prev in list] [next in list] [prev in thread] [next in thread] 

List:       sssd-devel
Subject:    [SSSD]Re:  [PATCH] TESTS: ldap_id_cleanup timeouts
From:       Michal Židek <mzidek () redhat ! com>
Date:       2015-08-21 11:33:09
Message-ID: 55D70C75.3020706 () redhat ! com
[Download RAW message or body]

On 08/20/2015 01:50 PM, Petr Cech wrote:
> On 08/19/2015 08:26 PM, Michal =C5=BDidek wrote:
>> Hi!
>>
>> This is another patch to avoid failing tests
>> in the CI (make-check-valgrind). This time
>> the ldap_id_cleanup tests.
>>
>> Looks like the one second cache timeout was too short
>> and the tests sometimes failed because they expected the
>> entries to be still valid for a short while
>> after they were added to sysdb.
>>
>> I saw the failures only on Fedora 20 CI machine.
>>
>> See the attached patch.
>>
>> Michal
> Hi,
> I just run your patch on my F22 VM and I see some trouble here... see
> attachment.
> Petr
> PS: I used clean GIT and your patch, nothing else. I know that this
> problem is another then you solved. But it is still issue.
>

I just saw 2 more fails in the CI because of the short
cache timeout. The problem you see, as you said as well,
is a different one and I agree it should be solved as
well but so far we were able to reproduce it on your
computer only and I did not see fails in the CI because of
that problem. I would suggest pushing this patch (if you
ACK it that is) to fix CI and look at the problem you
found later.

Michal
[prev in list] [next in list] [prev in thread] [next in thread]