Remove magic sleep from create_index_task

11 years ago 5ad91a0781 added a magic sleep to work around a rare deadlock
bug in memberOf plugin. Thierry is not aware of any outstanding issues
with memberOf plugin that could lead to a deadlock.

Related: https://pagure.io/freeipa/issue/8521
Signed-off-by: Christian Heimes <cheimes@redhat.com>
Reviewed-By: Rob Crittenden <rcritten@redhat.com>
This commit is contained in:
Christian Heimes
2020-09-21 16:25:53 +02:00
parent 37a0af6a8c
commit daec804961

View File

@@ -511,10 +511,6 @@ class LDAPUpdate:
def create_index_task(self, *attributes): def create_index_task(self, *attributes):
"""Create a task to update an index for attributes""" """Create a task to update an index for attributes"""
# Sleep a bit to ensure previous operations are complete
time.sleep(5)
cn_uuid = uuid.uuid1() cn_uuid = uuid.uuid1()
# cn_uuid.time is in nanoseconds, but other users of LDAPUpdate expect # cn_uuid.time is in nanoseconds, but other users of LDAPUpdate expect
# seconds in 'TIME' so scale the value down # seconds in 'TIME' so scale the value down