Hi, I couldn't find Daniel to do the merge, and it seems this was blocking the entire CI from deploying pkgs to tested for a couple of days now, and we have potential regressions in VDSM we have to verify are fixed, so I had to merge this on my own to get the system working and not slip into the weekend w/o a fix.
So just FYI I merged the fix. On Wed, Jan 17, 2018 at 8:19 PM, Nir Soffer <[email protected]> wrote: > On Wed, Jan 17, 2018 at 7:15 PM Nir Soffer <[email protected]> wrote: > >> Thanks for reporting this issue, should be fixed by >> https://gerrit.ovirt.org/#/c/86489/ >> >> Can someone trigger build artifacts job to test this patch? >> >> Can I trigger build-artifacts job manually? >> > > Ok, found how to do this. > > The issue here is empty release_suffix - this happens only when > we build a release version from tag. This the reason we did > not find this issue before, and we cannot detect this issue > by running build-artifacts on each build, unless we add a special > test to tag the local checkout, and build from the tag. > > Daniel, lets merge it and push a new tag to check if the build > works. > > >> >> On Wed, Jan 17, 2018 at 6:39 PM Nir Soffer <[email protected]> wrote: >> >>> On Wed, Jan 17, 2018 at 5:41 PM Eyal Edri <[email protected]> wrote: >>> >>>> On Wed, Jan 17, 2018 at 5:38 PM, Eyal Edri <[email protected]> wrote: >>>> >>>>> >>>>> >>>>> On Wed, Jan 17, 2018 at 5:23 PM, Nir Soffer <[email protected]> >>>>> wrote: >>>>> >>>>>> >>>>>> >>>>>> On Wed, Jan 17, 2018 at 5:08 PM Barak Korren <[email protected]> >>>>>> wrote: >>>>>> >>>>>>> Hi Guys, >>>>>>> >>>>>>> I'm sure you are aware that 'ovirt-imageio' is currently failing to >>>>>>> build on FC27 and Rawhide. >>>>>>> >>>>>> >>>>>> Why are you sure? I never seen any build failure since we added the >>>>>> fc27 and fcraw builds. >>>>>> >>>>>> Please point us to failed builds. >>>>>> >>>>> >>>>> Latest one - http://jenkins.ovirt.org/job/ovirt-imageio_master_ >>>>> build-artifacts-fc27-x86_64/32/ >>>>> >>>> >>>> If it doesn't take too long, I think its worth adding building RPMs to >>>> check-patch as well, so you can catch such errors >>>> before the merge. >>>> >>> >>> This is certainly what we need to do. >>> >>> >>>> >>>> >>>>> >>>>> >>>>>> >>>>>> >>>>>>> >>>>>>> I'm not sure you get the implications tough. >>>>>>> >>>>>>> The 1st thing that the release change-queue is checking, before >>>>>>> running OST, is if all builds for all patches it is checking are >>>>>>> building successfully for all platforms that the project has jobs >>>>>>> for. >>>>>>> >>>>>>> This means that essentially no new 'ovirt-imageio' packages are >>>>>>> making >>>>>>> it into the tested and nightly repos for __any__ platform. >>>>>>> >>>>>>> Furthermore, with the way the system works for now, there is no >>>>>>> special handling for build failures, with means they are treated like >>>>>>> OST failures, which means the system runs an expensive bisection >>>>>>> search to find the failing patch. So when your project fails to >>>>>>> build, >>>>>>> it slows down checking for other project's patches. >>>>>>> >>>>>>> I ask that you please do one of the following: >>>>>>> 1. Fix the FC27 and RAWHIDE builds >>>>>>> 2. Remove the FC27 and RAWHIDE build jobs >>>>>>> 3. Turn the FC27 and RAWHIDE build jobs into check-merged jobs which >>>>>>> are not checked by the change-queue. >>>>>>> >>>>>>> -- >>>>>>> Barak Korren >>>>>>> RHV DevOps team , RHCE, RHCi >>>>>>> Red Hat EMEA >>>>>>> redhat.com | TRIED. TESTED. TRUSTED. | redhat.com/trusted >>>>>>> >>>>>> >>>>>> _______________________________________________ >>>>>> Infra mailing list >>>>>> [email protected] >>>>>> http://lists.ovirt.org/mailman/listinfo/infra >>>>>> >>>>>> >>>>> >>>>> >>>>> -- >>>>> >>>>> Eyal edri >>>>> >>>>> >>>>> MANAGER >>>>> >>>>> RHV DevOps >>>>> >>>>> EMEA VIRTUALIZATION R&D >>>>> >>>>> >>>>> Red Hat EMEA <https://www.redhat.com/> >>>>> <https://red.ht/sig> TRIED. TESTED. TRUSTED. >>>>> <https://redhat.com/trusted> >>>>> phone: +972-9-7692018 <+972%209-769-2018> >>>>> irc: eedri (on #tlv #rhev-dev #rhev-integ) >>>>> >>>> >>>> >>>> >>>> -- >>>> >>>> Eyal edri >>>> >>>> >>>> MANAGER >>>> >>>> RHV DevOps >>>> >>>> EMEA VIRTUALIZATION R&D >>>> >>>> >>>> Red Hat EMEA <https://www.redhat.com/> >>>> <https://red.ht/sig> TRIED. TESTED. TRUSTED. >>>> <https://redhat.com/trusted> >>>> phone: +972-9-7692018 <+972%209-769-2018> >>>> irc: eedri (on #tlv #rhev-dev #rhev-integ) >>>> >>> -- Eyal edri MANAGER RHV DevOps EMEA VIRTUALIZATION R&D Red Hat EMEA <https://www.redhat.com/> <https://red.ht/sig> TRIED. TESTED. TRUSTED. <https://redhat.com/trusted> phone: +972-9-7692018 irc: eedri (on #tlv #rhev-dev #rhev-integ)
_______________________________________________ Infra mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/infra
