<div dir="ltr">Not sure I understand what do you mean by Gerrit Headers.<div>Can you give examples? </div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 25, 2016 at 4:57 PM, Nir Soffer <span dir="ltr"><<a href="mailto:nsoffer@redhat.com" target="_blank">nsoffer@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, Nov 25, 2016 at 4:45 PM, Tomáš Golembiovský <<a href="mailto:tgolembi@redhat.com">tgolembi@redhat.com</a>> wrote:<br>
> Hi,<br>
><br>
> I've noticed that in vdsm repo the merged commits do not contain the<br>
> info headers added by Gerrit any more (Reviewed-by/Reviewed-on/etc.)<wbr>.<br>
><br>
> Is that intentional? If yes, what was the motivation behind this?<br>
><br>
> The change seem to have happened about 4 days ago. Sometime between the<br>
> following two commits:<br>
><br>
> * 505f5da API: Introduce getQemuImageInfo API. [Maor Lipchuk]<br>
> * 1c4a39c protocoldetector: Avoid unneeded getpeername() [Nir Soffer]<br>
<br>
</span>We switched vdsm to fast-forward 4 days ago, maybe this was unintended<br>
side effect of this change?<br>
<br>
The gerrit headers are very useful, please add back.<br>
<br>
Nir<br>
______________________________<wbr>_________________<br>
Infra mailing list<br>
<a href="mailto:Infra@ovirt.org">Infra@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/infra" rel="noreferrer" target="_blank">http://lists.ovirt.org/<wbr>mailman/listinfo/infra</a><br>
<br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div>Eyal Edri<br>Associate Manager</div><div>RHV DevOps<br>EMEA ENG Virtualization R&D<br>Red Hat Israel<br><br>phone: +972-9-7692018<br>irc: eedri (on #tlv #rhev-dev #rhev-integ)</div></div></div></div></div></div></div>
</div>