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

[Dan Kenigsberg]

On Thu, Mar 29, 2018 at 7:27 PM, Nir Soffer <nsoffer at redhat.com> wrote:
>
> On Thu, Mar 29, 2018 at 7:17 PM Dan Kenigsberg <danken at redhat.com> wrote:
>>
>> On Wed, Mar 28, 2018 at 8:08 PM, Nir Soffer <nsoffer at redhat.com> wrote:
>> > On Wed, Mar 28, 2018 at 10:15 AM Dan Kenigsberg <danken at 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.

That was fast. Time to mention
https://gerrit.ovirt.org/#/q/status:open+project:vdsm+branch:master+topic:storage-mailbox
?