On 04/18/2016 10:39 AM, Lukas Slebodnik wrote:
On (02/12/15 17:10), Michal Židek wrote:
> I saw some integration tests failures recently,
> and I think there is a race condition between the
> enumeration refresh timeout and the sleeps
> after some operations that wait for this timeout.
> SSSD fails to populate changes from LDAP in time
> and some asserts can fail because of this.
> So far I saw 4 tests to fail like this, which
> is already quite a lot.
> The attached patch modifies the timeout values
> and hopefully removes the issue.
I think I found alternative solution for intermittent
failures of ADD REMOVE test with enumeration.
Not sure if this is safer than my patch. I made the
timeouts bigger on purpose, so that we avoid
problems with machines under heavy load.
You decided to go the opposite direction by
making one of the timeouts shorter.
That being said, maybe your patch is better,
because if it does not fail even under heavy
load it will make the tests shorter.
Once the CI is up we can try 20 test
runs with your patch and if they do not
fail I will give you an ACK.
If they fail, we can still fallback to my patch.
Occurence of failure was increased recently therefore
please review attached patch.
I know that Jakub wanted/tried to rewrite test without enumeration.
But I was mentioned in this thread that testing on enumeration is valid
We would have to test the enumeration sooner or later anyway.