On Thu, Mar 29, 2018 at 7:17 PM Dan Kenigsberg <dan...@redhat.com> wrote:

> On Wed, Mar 28, 2018 at 8:08 PM, Nir Soffer <nsof...@redhat.com> wrote:
> > On Wed, Mar 28, 2018 at 10:15 AM Dan Kenigsberg <dan...@redhat.com>
> 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.
_______________________________________________
Devel mailing list
Devel@ovirt.org
http://lists.ovirt.org/mailman/listinfo/devel

Reply via email to