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