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