Wrong target milestone bugs should be blocked with -verified on the stable branches
Tal Nisan
tnisan at redhat.com
Mon Jan 30 13:36:40 UTC 2017
I will, I'm dedicated to make my life as a stable branch maintainer easier,
thanks for that ;)
On Mon, Jan 30, 2017 at 3:34 PM, Eyal Edri <eedri at redhat.com> wrote:
> +1, thanks.
>
> Please continue to let us know if we need to enforce other hooks more
> strictly as well :)
>
> On Mon, Jan 30, 2017 at 3:29 PM, Tal Nisan <tnisan at redhat.com> wrote:
>
>> Yeah I've noticed the hook identified the situation, I'm asking you to
>> make it more stricter, hence the email :)
>>
>> On Mon, Jan 30, 2017 at 3:22 PM, Eyal Edri <eedri at redhat.com> wrote:
>>
>>> It already WARN on it:
>>>
>>> - Check TM::#1416309::WARN, wrong target milestone: ovirt-4.2.0
>>>
>>> We can make it more strict and give VERIFY -1.
>>> Adding infra-support to open a ticket on it.
>>>
>>> We intentionally didn't enforce Verify as default since we wanted to
>>> give time to the hook
>>> to work and see we don't have false positives on it.
>>>
>>>
>>> On Mon, Jan 30, 2017 at 3:13 PM, Tal Nisan <tnisan at redhat.com> wrote:
>>>
>>>> Example:
>>>> https://gerrit.ovirt.org/#/c/71374/
>>>>
>>>> I'd expect the Gerrit hooks to give -1 on verified when a 4.2 bug is
>>>> pushed into the 4.1 stable branch
>>>>
>>>> Thanks.
>>>>
>>>> _______________________________________________
>>>> Infra mailing list
>>>> Infra at 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 <+972%209-769-2018>
>>> 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 <+972%209-769-2018>
> irc: eedri (on #tlv #rhev-dev #rhev-integ)
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/infra/attachments/20170130/c4229f22/attachment.html>
More information about the Infra
mailing list