<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Dec 11, 2016 at 4:00 PM, Barak Korren <span dir="ltr"><<a href="mailto:bkorren@redhat.com" target="_blank">bkorren@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 11 December 2016 at 11:18, Eyal Edri <<a href="mailto:eedri@redhat.com">eedri@redhat.com</a>> wrote:<br>
> Adding infra as well.<br>
><br>
> I see a very strange thing happening on the build artifacts jobs:<br>
><br>
> On [1] we see a successful build of 1.2.10 built in build-artifacts, but on<br>
> the 2 patches merged after it, its back to 1.2.9 [2].<br>
> Is it possible that the 2 patches merged after the version bump weren't<br>
> rebased on the version branch and were built using older code?<br>
<br>
</span>Is build_artifacts running before or after merge?<br>
This is impossible if its running after.<br>
<span class=""><br>
> The project has 'fast forward only' mode in Gerrit.<br>
<br>
</span>If build_artifacts runs on pre-merge code that even this will not help.<br>
<br>
Bottom line - build_artifacs should never ever run on unmerged patches.<br></blockquote><div><br></div><div>AFAIK all of these builds were triggered on merged patches. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
<br>
--<br>
Barak Korren<br>
<a href="mailto:bkorren@redhat.com">bkorren@redhat.com</a><br>
RHCE, RHCi, RHV-DevOps Team<br>
<a href="https://ifireball.wordpress.com/" rel="noreferrer" target="_blank">https://ifireball.wordpress.<wbr>com/</a><br>
</font></span></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></div>