Re: ovirt-engine-4.0.5 is not treated as a monitored stable branch

I think its this one: https://gerrit.ovirt.org/#/c/64538/ And I think I know what the problem is, This bug [1] is 4.1 bug its target milestone is 4.0.6 because its a clone candidate bug. Shlomi - IIRC we addressed that issue in the hook, can you verify? (also, might be the bug is ignoring downstream bugs) [1] https://bugzilla.redhat.com/show_bug.cgi?id=1368101 On Sun, Oct 30, 2016 at 3:35 PM, Eyal Edri <eedri(a)redhat.com&gt; wrote: > Shlomi - do we have a gerrit hook that checks the bug version with the > branch name? > I remember we had something similar... > > On Sun, Oct 30, 2016 at 3:30 PM, Tal Nisan <tnisan(a)redhat.com&gt; wrote: > >> That was an example, the patch should have got -verified as a 4.1 bug >> url in a 4.0.5 branch >> >> On Sun, Oct 30, 2016 at 3:27 PM, Shlomo Ben David <sbendavi(a)redhat.com&gt; >> wrote: >> >>> Hi, >>> >>> Tal: The '*Check merged to previous*' hook is checking if the current >>> patch was merged to previous stable branches and i think that it is not >>> related to the one that you're searching. >>> >>> Eyal: Currently there is no hook that checks if branch is enabled, but >>> there is hook that checks if the patch major version branch is [equal / >>> higher / lower] than the bug target milestone major version. >>> i'll explain by giving you few examples and their results: >>> >>> Example 1: >>> tm: '---' (empty target milestone) >>> branch major version: any >>> result: *WARN MSG* >>> >>> Example 2: >>> tm major version: 3.6 >>> branch major version: 3.6 >>> result: *OK MSG* >>> >>> Example 3: >>> tm major version: 3.6 >>> branch major version: 4.0 >>> result: checks for clone flags >>> (if tm major version in flags - *OK MSG* else *WARN MSG*) >>> >>> Example 4: >>> tm major version: 4.0 >>> branch major version: 3.6 >>> result: *WARN MSG* >>> >>> >>> Best Regards, >>> >>> Shlomi Ben-David | DevOps Engineer | Red Hat ISRAEL >>> RHCSA | RHCE >>> IRC: shlomibendavid (on #rhev-integ, #rhev-dev, #rhev-ci) >>> >>> OPEN SOURCE - 1 4 011 && 011 4 1 >>> >>> On Sun, Oct 30, 2016 at 9:45 AM, Eyal Edri <eedri(a)redhat.com&gt; wrote: >>> >>>> Shlomi, >>>> Can you check if we have the hook for verifying right branch is >>>> enabled? >>>> >>>> >>>> >>>> On Thu, Oct 27, 2016 at 11:00 AM, Tal Nisan <tnisan(a)redhat.com&gt; wrote: >>>> >>>>> This patch was pushed to 4.0.5 with a bug that at the time of the >>>>> push was 4.1, Gerrit hooks should have marked it as -verified, instead you >>>>> get: >>>>> >>>>> - Check merged to previous::IGNORE, Not in stable branch >>>>> (['ovirt-engine-3.6', 'ovirt-engine-4.0', 'ovirt-engine-3.6.9', >>>>> 'ovirt-engine-4.0.2', 'ovirt-engine-4.0.4']) >>>>> >>>>> >>>>> On Thu, Oct 27, 2016 at 9:55 AM, Gil Shinar <gshinar(a)redhat.com&gt; >>>>> wrote: >>>>> >>>>>> Can you please elaborate? >>>>>> >>>>>> Thanks >>>>>> Gil >>>>>> >>>>>> On Wed, Oct 26, 2016 at 11:50 AM, Tal Nisan <tnisan(a)redhat.com&gt; >>>>>> wrote: >>>>>> >>>>>>> This patch for example: >>>>>>> https://gerrit.ovirt.org/#/c/65398/ >>>>>>> >>>>>>> >>>>>>> _______________________________________________ >>>>>>> Infra mailing list >>>>>>> Infra(a)ovirt.org >>>>>>> http://lists.ovirt.org/mailman/listinfo/infra >>>>>>> >>>>>>> >>>>>> >>>>> >>>>> _______________________________________________ >>>>> Infra mailing list >>>>> Infra(a)ovirt.org >>>>> http://lists.ovirt.org/mailman/listinfo/infra >>>>> >>>>> >>>> >>>> >>>> -- >>>> Eyal Edri >>>> Associate Manager >>>> RHV DevOps >>>> EMEA ENG Virtualization R&D >>>> Red Hat Israel >>>> >>>> phone: +972-9-7692018 >>>> irc: eedri (on #tlv #rhev-dev #rhev-integ) >>>> >>> >>> >> > > > -- > Eyal Edri > Associate Manager > RHV DevOps > EMEA ENG Virtualization R&D > Red Hat Israel > > phone: +972-9-7692018 > irc: eedri (on #tlv #rhev-dev #rhev-integ) > -- Eyal Edri Associate Manager RHV DevOps EMEA ENG Virtualization R&D Red Hat Israel phone: +972-9-7692018 irc: eedri (on #tlv #rhev-dev #rhev-integ)