<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Wed, Mar 28, 2018 at 10:15 AM Dan Kenigsberg <<a href="mailto:danken@redhat.com">danken@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I suppose this test should be reworked, or at least, have its constants tweaked.<br>
<br>
FAIL: test_slow_handler_sync (common.logutils_test.TestThreadedHandler)<br>
----------------------------------------------------------------------<br>
Traceback (most recent call last):<br>
File "/home/jenkins/workspace/vdsm_master_check-patch-fc27-x86_64/vdsm/tests/common/logutils_test.py",<br>
line 319, in test_slow_handler_sync<br>
self.assertGreater(max(workers_time), 0.9)<br>
AssertionError: 0.8714249134063721 not greater than 0.9<br></blockquote><div><br></div><div>I tried to fix this last week, but for some reason the fix does not help on the CI.</div><div><br></div><div>I think we can remove this test, it is reproducing the issues with the standard </div><div>logging code, and does not test the new code.</div><div><br></div><div>Nir</div><div> </div></div></div>