Il giorno ven 22 mar 2019 alle ore 11:14 Dan Kenigsberg <dan...@redhat.com> ha scritto:
> > > On Fri, 22 Mar 2019, 12:00 Sandro Bonazzola, <sbona...@redhat.com> wrote: > >> >> >> Il giorno ven 22 mar 2019 alle ore 10:52 Dan Kenigsberg < >> dan...@redhat.com> ha scritto: >> >>> Yes, I'm repeating myself. >>> SKIPPING TESTS IS BAD >>> >> >> I agree. And having the suite failing on a broken test skipping all the >> following tests is even worse. >> This is why I would prefer the rest of the product being tested while >> someone take ownership of the broken test and fix it. >> > > This is a good reason to rewrite OST with pytest, which continues on > failure. > Patches are welcome :-) > And a good reason to ping mperina on IRC to debug this. And a good reason > not to merge new code. > > It doesn't convince me that we should ignore the failure without due > debugging. > Debugging in indeed needed but not on production system blocking the rest of the CI. Maintainer of the test can debug it on own test environment. > > >> >> >>> >>> We have a test suite in order to fix bugs, not in order to kill itself. >>> >>> Host hooks are Infra. Infra is mperina, rnori and msobczik. >>> Please consult with them before you shut our collective eyes. >>> >>> Please point them to a failing job, and record the failing traceback. >>> >>> On Fri, 22 Mar 2019, 11:27 Dafna Ron, <d...@redhat.com> wrote: >>> >>>> patch submitted: https://gerrit.ovirt.org/#/c/98773/ >>>> >>>> Thanks, >>>> Dafna >>>> >>>> >>>> On Fri, Mar 22, 2019 at 9:04 AM Sandro Bonazzola <sbona...@redhat.com> >>>> wrote: >>>> >>>>> >>>>> >>>>> Il giorno ven 22 mar 2019 alle ore 09:34 Dafna Ron <d...@redhat.com> >>>>> ha scritto: >>>>> >>>>>> Hi, >>>>>> >>>>>> we are randomly failing on get_host_hooks test for at least 3 weeks. >>>>>> its not a specific branch or project and there are no commonalities >>>>>> that I can see, aside from not being able to communicate with the host. >>>>>> >>>>>> this week its started happening at least once a day (this morning, 2 >>>>>> out of 3 failures were due to that test). >>>>>> >>>>>> This test has been added by Yaniv Kaul over a year ago and he is no >>>>>> longer working on ovirt I think someone else should take ownership of >>>>>> this >>>>>> test and fix it. >>>>>> Please let me know if you are intending to investigate and either fix >>>>>> the failure or fix the test if not I will add a skip to the test, >>>>>> >>>>> >>>>> Please add a skip to the test and if someone will step in maintaining >>>>> this test it will be re-enabled. >>>>> >>>>> >>>>> >>>>>> >>>>>> Thanks, >>>>>> Dafna >>>>>> >>>>>> >>>>>> >>>>> >>>>> -- >>>>> >>>>> SANDRO BONAZZOLA >>>>> >>>>> MANAGER, SOFTWARE ENGINEERING, EMEA R&D RHV >>>>> >>>>> Red Hat EMEA <https://www.redhat.com/> >>>>> >>>>> sbona...@redhat.com >>>>> <https://red.ht/sig> >>>>> >>>> >> >> -- >> >> SANDRO BONAZZOLA >> >> MANAGER, SOFTWARE ENGINEERING, EMEA R&D RHV >> >> Red Hat EMEA <https://www.redhat.com/> >> >> sbona...@redhat.com >> <https://red.ht/sig> >> > -- SANDRO BONAZZOLA MANAGER, SOFTWARE ENGINEERING, EMEA R&D RHV Red Hat EMEA <https://www.redhat.com/> sbona...@redhat.com <https://red.ht/sig>
_______________________________________________ Devel mailing list -- devel@ovirt.org To unsubscribe send an email to devel-le...@ovirt.org Privacy Statement: https://www.ovirt.org/site/privacy-policy/ oVirt Code of Conduct: https://www.ovirt.org/community/about/community-guidelines/ List Archives: https://lists.ovirt.org/archives/list/devel@ovirt.org/message/O72UUDT3CYPOILF4OPLFXBCPH3PJQD2Z/