<div dir="ltr">Sorry, my conclusion was too hasty.<br>Juan, you're right - that's my patch fault. All failures I saw were caused by a single faulty patch of mine. :-(<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jun 20, 2016 at 7:08 PM, Juan Hernández <span dir="ltr"><<a href="mailto:jhernand@redhat.com" target="_blank">jhernand@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 class="HOEnZb"><div class="h5">On 06/20/2016 06:02 PM, Yevgeny Zaspitsky wrote:<br>
> Hi All,<br>
><br>
> It looks like since [1] commit was merged to master the [2] became stale<br>
> as it refers to findbugs-general maven profile that doesn't exist anymore.<br>
><br>
> Infra team, please update [2] asap, as it faulty marks all patches as -1.<br>
><br>
> [1] <a href="https://gerrit.ovirt.org/#/c/48687" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/48687</a><br>
> [2] <a href="http://jenkins.ovirt.org/job/ovirt-engine_master_find-bugs_gerrit" rel="noreferrer" target="_blank">http://jenkins.ovirt.org/job/ovirt-engine_master_find-bugs_gerrit</a><br>
><br>
> Regards,<br>
> Yevgeny<br>
<br>
</div></div>The fact that the job uses a profile that doesn't exist shouldn't be<br>
problematic, it will just cause a warning. It would be good to remove<br>
that "-Pfindbugs-general" option anynow, to avoid the warning.<br>
<br>
I see the job running correctly for many patches. Do you have an example<br>
of a patch where it voted -1 incorrectly?<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Dirección Comercial: C/Jose Bardasano Baos, 9, Edif. Gorbea 3, planta<br>
3ºD, 28016 Madrid, Spain<br>
Inscrita en el Reg. Mercantil de Madrid – C.I.F. B82657941 - Red Hat S.L.<br>
</font></span></blockquote></div><br></div>