Re: [ovirt-devel] new time-based test randomly failing

On Thu, Mar 29, 2018 at 7:17 PM Dan Kenigsberg <danken(a)redhat.com&gt; wrote: > > On Wed, Mar 28, 2018 at 8:08 PM, Nir Soffer <nsoffer(a)redhat.com&gt; wrote: > > On Wed, Mar 28, 2018 at 10:15 AM Dan Kenigsberg <danken(a)redhat.com&gt; > > wrote: > >> > >> I suppose this test should be reworked, or at least, have its constants > >> tweaked. > >> > >> FAIL: test_slow_handler_sync (common.logutils_test.TestThreadedHandler) > >> ---------------------------------------------------------------------- > >> Traceback (most recent call last): > >> File > >> > >> "/home/jenkins/workspace/vdsm_master_check-patch-fc27-x86_64/vdsm/tests/common/logutils_test.py", > >> line 319, in test_slow_handler_sync > >> self.assertGreater(max(workers_time), 0.9) > >> AssertionError: 0.8714249134063721 not greater than 0.9 > > > > > > I tried to fix this last week, but for some reason the fix does not help > > on > > the CI. > > > > I think we can remove this test, it is reproducing the issues with the > > standard > > logging code, and does not test the new code. > > Ok: https://gerrit.ovirt.org/89618 Thanks, akced.