<div dir="ltr">Oh I see now what you are talking about, indeed in patchset 20 jobs were marked as successful even though the actually failed, a bug indeed</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 19, 2016 at 5:33 PM, Martin Perina <span dir="ltr"><<a href="mailto:mperina@redhat.com" target="_blank">mperina@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_extra"><br><div class="gmail_quote"><span class="">On Thu, May 19, 2016 at 4:26 PM, Tal Nisan <span dir="ltr"><<a href="mailto:tnisan@redhat.com" target="_blank">tnisan@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">No need, already reverted once I've read you message, had it reverted and verified so I could keep working on the branch - <a href="https://gerrit.ovirt.org/#/c/57742/" target="_blank">https://gerrit.ovirt.org/#/c/57742/</a><div>BTW, both jobs ended with a failure:</div><div><p style="white-space:pre-wrap;color:rgb(53,53,53);font-family:sans-serif"><a href="http://jenkins.ovirt.org/job/ovirt-engine_master_upgrade-from-master_el7_merged/310/" style="color:rgb(6,84,172);text-decoration:none" target="_blank">http://jenkins.ovirt.org/job/ovirt-engine_master_upgrade-from-master_el7_merged/310/</a> : FAILURE</p><p style="white-space:pre-wrap;color:rgb(53,53,53);font-family:sans-serif"><a href="http://jenkins.ovirt.org/job/ovirt-engine_master_upgrade-from-3.6_el7_merged/307/" style="color:rgb(6,84,172);text-decoration:none" target="_blank">http://jenkins.ovirt.org/job/ovirt-engine_master_upgrade-from-3.6_el7_merged/307/</a> : FAILURE</p></div></div></blockquote></span><div><br><div class="gmail_default" style="font-family:arial,helvetica,sans-serif;display:inline">Yes, but those are executed only after patch is merged, check-patch jobs are return SUCCESS if when build failed, David is working on that.<br><br></div></div><div><div class="h5"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><p style="white-space:pre-wrap;color:rgb(53,53,53);font-family:sans-serif"><br></p></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 19, 2016 at 5:22 PM, David Caro <span dir="ltr"><<a href="mailto:dcaro@redhat.com" target="_blank">dcaro@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>On 05/19 16:20, Martin Perina wrote:<br>
> On Thu, May 19, 2016 at 4:15 PM, David Caro <<a href="mailto:dcaro@redhat.com" target="_blank">dcaro@redhat.com</a>> wrote:<br>
><br>
> > On 05/19 16:14, Martin Perina wrote:<br>
> > > Hi,<br>
> > ><br>
> > > so I probably found the issue why this broke the build. I have checked<br>
> > the<br>
> > > CI before merging and it was OK,<br>
> > > see CI+1 on patch set 20.<br>
> > ><br>
> > > Now looking at console outputs of those jobs, the build itself failed,<br>
> > but<br>
> > > build jobs are marked as SUCCESSFULL.<br>
> > > So it seems we have a bug in those jobs!!!<br>
> ><br>
> ><br>
> > We will need a bit more info on which project/jenkins job/build, etc, by<br>
> > the<br>
> > email I can't extract the project that failed<br>
> ><br>
><br>
> Sorry, here is the problematic patch: <a href="https://gerrit.ovirt.org/#/c/57052/" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/57052/</a><br>
><br>
> The problematic patch is build on patch set 20. It's marked as successfull,<br>
> although engine build failed:<br>
><br>
> <a href="http://jenkins.ovirt.org/job/ovirt-engine_master_check-patch-el7-x86_64/1250/console" rel="noreferrer" target="_blank">http://jenkins.ovirt.org/job/ovirt-engine_master_check-patch-el7-x86_64/1250/console</a><br>
><br>
><br>
> Anyway, we are preparing reverting patch ...<br>
<br>
<br>
</span>I see, the issue was introduced by me in the mock_runner.sh script, will send a<br>
patch right away<br>
<div><div><br>
><br>
> Sorry for the issue.<br>
><br>
> Martin<br>
> <br>
><br>
><br>
> ><br>
> > ><br>
> > ><br>
> > ><br>
> > ><br>
> > ><br>
> > > On Thu, May 19, 2016 at 3:53 PM, Tal Nisan <<a href="mailto:tnisan@redhat.com" target="_blank">tnisan@redhat.com</a>> wrote:<br>
> > ><br>
> > > > This commit broke the build, missing KernelEnv.<br>
> > > > Normally I'd revert but since it's feature freeze I'm more forgiving :)<br>
> > > > Please either revert or send a fix asap.<br>
> > > ><br>
> > > > Thanks.<br>
> > > ><br>
> > > ><br>
> ><br>
> > > _______________________________________________<br>
> > > Infra mailing list<br>
> > > <a href="mailto:Infra@ovirt.org" target="_blank">Infra@ovirt.org</a><br>
> > > <a href="http://lists.ovirt.org/mailman/listinfo/infra" rel="noreferrer" target="_blank">http://lists.ovirt.org/mailman/listinfo/infra</a><br>
> ><br>
> ><br>
> > --<br>
> > David Caro<br>
> ><br>
> > Red Hat S.L.<br>
> > Continuous Integration Engineer - EMEA ENG Virtualization R&D<br>
> ><br>
> > Tel.: <a href="tel:%2B420%20532%20294%20605" value="+420532294605" target="_blank">+420 532 294 605</a><br>
> > Email: <a href="mailto:dcaro@redhat.com" target="_blank">dcaro@redhat.com</a><br>
> > IRC: dcaro|dcaroest@{freenode|oftc|redhat}<br>
> > Web: <a href="http://www.redhat.com" rel="noreferrer" target="_blank">www.redhat.com</a><br>
> > RHT Global #: 82-62605<br>
> ><span><font color="#888888"><br>
<br>
--<br>
David Caro<br>
<br>
Red Hat S.L.<br>
Continuous Integration Engineer - EMEA ENG Virtualization R&D<br>
<br>
Tel.: <a href="tel:%2B420%20532%20294%20605" value="+420532294605" target="_blank">+420 532 294 605</a><br>
Email: <a href="mailto:dcaro@redhat.com" target="_blank">dcaro@redhat.com</a><br>
IRC: dcaro|dcaroest@{freenode|oftc|redhat}<br>
Web: <a href="http://www.redhat.com" rel="noreferrer" target="_blank">www.redhat.com</a><br>
RHT Global #: 82-62605<br>
</font></span></div></div></blockquote></div><br></div>
</blockquote></div></div></div><br></div></div>
</blockquote></div><br></div>