389ds / 389-ds-base

The enterprise-class Open Source LDAP server for Linux
https://www.port389.org/
Other
200 stars 82 forks source link

Issue 6233 - CI test wait_for_async_feature_test sometime fails #6234

Closed progier389 closed 1 week ago

progier389 commented 2 weeks ago

CI test random failure related to timing. Fixed by decreasing the minimum number of expected asynchronous results

Issue: #6233

Reviewed by: @droideck, @tbordaz

tbordaz commented 2 weeks ago

LGTM as well I was wondering how it works ? It counts the number of 'waitfor_async_results' message depending on the 'nsds5ReplicaWaitForAsyncResults' tuning. Fine. Now in 'test_behavior_with_value' it setup a 2 suppliers topology. Enable repl logs then waits 20s to see the final count. What is the guarantee that the agreement did connect ? Shouldn't it do an dummy update after setting the repl_log ?