[ovirt-devel] Gerrit headers are not added to commits in vdsm repo

Eyal Edri eedri at redhat.com
Sun Nov 27 10:31:21 UTC 2016


Not sure I understand what do you mean by Gerrit Headers.
Can you give examples?

On Fri, Nov 25, 2016 at 4:57 PM, Nir Soffer <nsoffer at redhat.com> wrote:

> On Fri, Nov 25, 2016 at 4:45 PM, Tomáš Golembiovský <tgolembi at redhat.com>
> wrote:
> > Hi,
> >
> > I've noticed that in vdsm repo the merged commits do not contain the
> > info headers added by Gerrit any more (Reviewed-by/Reviewed-on/etc.).
> >
> > Is that intentional? If yes, what was the motivation behind this?
> >
> > The change seem to have happened about 4 days ago. Sometime between the
> > following two commits:
> >
> > * 505f5da  API: Introduce getQemuImageInfo API. [Maor Lipchuk]
> > * 1c4a39c  protocoldetector: Avoid unneeded getpeername() [Nir Soffer]
>
> We switched vdsm to fast-forward 4 days ago, maybe this was unintended
> side effect of this change?
>
> The gerrit headers are very useful, please add back.
>
> Nir
> _______________________________________________
> 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
irc: eedri (on #tlv #rhev-dev #rhev-integ)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/devel/attachments/20161127/e62976db/attachment-0001.html>


More information about the Devel mailing list