On 08/21/2015 01:33 PM, Michal Židek wrote:
On 08/20/2015 01:50 PM, Petr Cech wrote:
> On 08/19/2015 08:26 PM, Michal Židek 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
OK, I agree.
There is new ticket about the mentioned bug:
https://fedorahosted.org/sssd/ticket/2768
And there are CI tests:
http://sssd-ci.duckdns.org/logs/job/23/57/summary.html
(The failing is not connected to this patch.)
ACK
Petr