Change in ovirt-engine[master]: packaging: engine-backup: Fix for postgresql 8

Eyal Edri eedri at redhat.com
Tue May 19 14:21:12 UTC 2015



----- Original Message -----
> From: "David Caro" <dcaroest at redhat.com>
> To: "Itamar Heim" <iheim at redhat.com>
> Cc: "Yedidyah Bar David" <didi at redhat.com>, "infra" <infra at ovirt.org>, "Eyal Edri" <eedri at 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 at ovirt.org
> > >>To: "Sandro Bonazzola" <sbonazzo at redhat.com>, "Yedidyah Bar David"
> > >><didi at redhat.com>
> > >>Cc: "Simone Tiraboschi" <stirabos at 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 at 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 at redhat.com>
> > >>Gerrit-Reviewer: Jenkins CI
> > >>Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
> > >>Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
> > >>Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>
> > >>Gerrit-Reviewer: automation at 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 at redhat.com
> Web: www.redhat.com
> RHT Global #: 82-62605
> 



More information about the Infra mailing list