URL:
https://github.com/SSSD/sssd/pull/947
Title: #947: tests: fix race conditions in integration tests
alexey-tikhonov commented:
"""
> tl,dr: I'm not categorically against estimation of
REAL_ENUMERATION_TIMEOUT as 2*ENUMERATION_TIMEOUT.
> But IMO this has nothing to do with Nyquist frequency and this is still arbitrary
(just "good enough with a very good margin") value.
You did not give me any argument to convince me otherwise.
Hm... Not sure how could I convince you that I just do not understand how it is relevant
:) (which doesn't necessarily mean it is not, I really might miss something)
I can remove this name from comments and code if you feel strongly
about it.
I would opt to either (preferably) explain why "*2" is sort of
special/"guaranteed" safe value (i.e. how sampling rate/high freq applicable
here) or stick with "good enough margin" explanation. I am fine with both.
I don't care that it will be five minutes slower if it runs
correctly.
Then we could just change /2 -> 1 and push test_enumeration patch.
Case of files provider is more complicated IMO.
"""
See the full comment at
https://github.com/SSSD/sssd/pull/947#issuecomment-559469560