URL: https://github.com/SSSD/sssd/pull/956
Title: #956: tests: fix race confition in enumeration tests

alexey-tikhonov commented:
"""
Hi @mzidek-rh,

> I agree with the reasoning for this patch. ACK.

Do you by any chance have an explanation for the this 
`time.sleep(INTERACTIVE_TIMEOUT/2)` (replaced with 
`time.sleep(INTERACTIVE_TIMEOUT)` in this patch)?

Commit message is talking about "the time we should wait in order to have the 
changes reflected" but that's not the case for this sleep, IMO.
"""

See the full comment at 
https://github.com/SSSD/sssd/pull/956#issuecomment-562154615
_______________________________________________
sssd-devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedorahosted.org/archives/list/[email protected]

Reply via email to