----- Original Message -----
From: "David Caro" <dcaroest(a)redhat.com>
To: "Itamar Heim" <iheim(a)redhat.com>
Cc: "Yedidyah Bar David" <didi(a)redhat.com>, "infra"
<infra(a)ovirt.org>, "Eyal Edri" <eedri(a)redhat.com>
Sent: Tuesday, May 19, 2015 5:16:07 PM
Subject: Re: Change in ovirt-engine[master]: packaging: engine-backup: Fix for postgresql
8
On 05/19, Itamar Heim wrote:
> On 05/19/2015 05:00 PM, Yedidyah Bar David wrote:
> >----- Original Message -----
> >>From: automation(a)ovirt.org
> >>To: "Sandro Bonazzola" <sbonazzo(a)redhat.com>, "Yedidyah
Bar David"
> >><didi(a)redhat.com>
> >>Cc: "Simone Tiraboschi" <stirabos(a)redhat.com>
> >>Sent: Tuesday, May 19, 2015 4:50:18 PM
> >>Subject: Change in ovirt-engine[master]: packaging: engine-backup: Fix
> >>for postgresql 8
> >>
> >>automation(a)ovirt.org has posted comments on this change.
> >>
> >>Change subject: packaging: engine-backup: Fix for postgresql 8
> >>......................................................................
> >>
> >>
> >>Patch Set 2:
> >>
> >>* Update tracker::#1222553::OK
> >>* Check TR::#1222553::ERROR, 3.6.0 should not match .*
> >
> >Any idea?
> >
> >Bug's target release is 3.6.0, patch on master branch.
> >
> >David says the message says that the bug should not have a target release.
>
> not sure, david?
not sure either, if that's not the desired behavior I can change it (it's
been
some time since the last change).
i think iirc we're not enforcing master to have bug-urls, and the gerrit hook
doesn't find any 3.6.* branches (since we didn't branch yet) hence throws a
warning.
i think we should update the hook to recognize master as 3.6 and warn when a bug-url
isn't on the patch.
>
> >
> >Why?
> >
> >>
> >>--
> >>To view, visit
https://gerrit.ovirt.org/41118
> >>To unsubscribe, visit
https://gerrit.ovirt.org/settings
> >>
> >>Gerrit-MessageType: comment
> >>Gerrit-Change-Id: Ia1164822326ad117e960309f914450b4eefb2c98
> >>Gerrit-PatchSet: 2
> >>Gerrit-Project: ovirt-engine
> >>Gerrit-Branch: master
> >>Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com>
> >>Gerrit-Reviewer: Jenkins CI
> >>Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
> >>Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
> >>Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
> >>Gerrit-Reviewer: automation(a)ovirt.org
> >>Gerrit-HasComments: No
> >>
> >
>
--
David Caro
Red Hat S.L.
Continuous Integration Engineer - EMEA ENG Virtualization R&D
Tel.: +420 532 294 605
Email: dcaro(a)redhat.com
Web:
www.redhat.com
RHT Global #: 82-62605