On Tuesday, September 16, 2014 11:22:11 AM Martin Sivak wrote:
Well Related-To means exactly what you are proposing where I came
from (the
platform group, distgit, errata..) :)
But whatever suits you.
Martin
--
Martin Sivák
msivak(a)redhat.com
Red Hat Czech
RHEV-M SLA / Brno, CZ
----- Original Message -----
> ----- Original Message -----
>
> > From: "Martin Sivak" <msivak(a)redhat.com>
> > To: "Nir Soffer" <nsoffer(a)redhat.com>
> > Cc: "David Caro" <dcaroest(a)redhat.com>, devel(a)ovirt.org
> > Sent: Tuesday, September 16, 2014 3:53:53 PM
> > Subject: Re: [ovirt-devel] Stable branch gerrit hook and Related-To: bug
> >
> > > What if gerrit used the dependency info and would not -1 patches
> > > that are dependency of a patch with a bug url?
> >
> > That is not a bad idea, but there is one use case you do not consider..
> > it
> > would mean that feature cannot be merged before it is completed.
> >
> > I like my patches to be merged as soon as possible. Of course if the
> > patches
> > leave the tree in working state.
>
> Right
>
> > The patchset that started this thread already depends on about 15
> > patches
> > that were merged to master two moths ago.
>
> So how about:
>
> Required-For:
https://bugzilla.redhat.com/1096197
>
> You must merge this to get the bug fixed, but the bug status is not
> changed when merged.
>
> The hook can add the patch to the bug as a tracker.
>
> David, can we easily add such tag?
No problem, but can you please summarize what needs to be done for each?
If you open a track ticket with that info even better :)
- When patch created (move bug to POST, add external tracker, other....)
- When patch merged (move bug to post if not more open external patches)
- Same checks that with Bug-Url (good product, target release for stable
branch, public bug, ...)
* About the Related-To tag
* About the Required-For tag
Thanks!
>
> NIr
--
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