<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Nov 23, 2016 at 10:26 AM, Juan Hernández <span dir="ltr">&lt;<a href="mailto:jhernand@redhat.com" target="_blank">jhernand@redhat.com</a>&gt;</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/23/2016 09:16 AM, Eyal Edri wrote:<br>
&gt; Maybe worth adding the build artifacts to check-patch.sh so you&#39;ll see<br>
&gt; it before merge?<br>
&gt;<br>
<br>
</span>The check-patch.sh script already runs the build of the artifacts. The<br>
problem is that in this case the &quot;bug&quot; was in the automation<br>
build-artifacts.sh script itself, and it only runs when the patch is<br>
merged. The reason for running the build-artifacts.sh script only when<br>
the patch is merged is that it runs a heavy process: generation of<br>
documentation using Publican. I didn&#39;t want to do that for every patch,<br>
as it consumes a lot of resources and doesn&#39;t add anything in terms of<br>
patch verification.<br></blockquote><div><br></div><div>Gating will help with that, we&#39;re working on it now, you can be a beta tested for the api project if you want :)</div><div>So basically you&#39;ll have a flag in gerrit to trigger the pre-merge jobs and if the job will fail the patch won&#39;t merge.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Actually I am currently using the build-artifacts.sh script where I<br>
should probably use the check-merged.sh script. I think I need to<br>
reorganize the jobs and the automation scripts to make this clearer. I<br>
am working on that.<br>
<span class=""><br>
&gt; On Wed, Nov 23, 2016 at 10:10 AM, Juan Hernández &lt;<a href="mailto:jhernand@redhat.com">jhernand@redhat.com</a><br>
</span><span class="">&gt; &lt;mailto:<a href="mailto:jhernand@redhat.com">jhernand@redhat.com</a>&gt;&gt; wrote:<br>
&gt;<br>
&gt;     On 11/23/2016 08:29 AM, Sandro Bonazzola wrote:<br>
&gt;     &gt;<br>
&gt;     <a href="http://jenkins.ovirt.org/job/ovirt-engine-api-model_4.0_build-artifacts-fc24-x86_64/30/console" rel="noreferrer" target="_blank">http://jenkins.ovirt.org/job/<wbr>ovirt-engine-api-model_4.0_<wbr>build-artifacts-fc24-x86_64/<wbr>30/console</a><br>
&gt;     &lt;<a href="http://jenkins.ovirt.org/job/ovirt-engine-api-model_4.0_build-artifacts-fc24-x86_64/30/console" rel="noreferrer" target="_blank">http://jenkins.ovirt.org/job/<wbr>ovirt-engine-api-model_4.0_<wbr>build-artifacts-fc24-x86_64/<wbr>30/console</a>&gt;<br>
&gt;     &gt;<br>
&gt;     &gt; *<br>
&gt;     &gt; *<br>
&gt;     &gt;<br>
&gt;     &gt; *<br>
&gt;     &gt; *<br>
&gt;     &gt;<br>
&gt;     &gt; *00:04:35.993* + mv target/model.json exported-artifacts<br>
&gt;     &gt; *00:04:35.993* mv: cannot stat &#39;target/model.json&#39;: No such file<br>
&gt;     or directory<br>
&gt;     &gt; *00:04:35.993* Took 93 seconds<br>
&gt;     &gt;<br>
&gt;     &gt;<br>
&gt;     &gt; Can you please have a look?<br>
&gt;     &gt;<br>
&gt;<br>
&gt;<br>
&gt;     That should be addressed by the following patch:<br>
&gt;<br>
&gt;       Fix automation scripts after removing &#39;describe&#39; profile<br>
&gt;       <a href="https://gerrit.ovirt.org/67208" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/67208</a><br>
&gt;<br>
&gt;     --<br>
&gt;     Dirección Comercial: C/Jose Bardasano Baos, 9, Edif. Gorbea 3, planta<br>
&gt;     3ºD, 28016 Madrid, Spain<br>
&gt;     Inscrita en el Reg. Mercantil de Madrid – C.I.F. B82657941 - Red Hat<br>
&gt;     S.L.<br>
&gt;     ______________________________<wbr>_________________<br>
&gt;     Infra mailing list<br>
</span>&gt;     <a href="mailto:Infra@ovirt.org">Infra@ovirt.org</a> &lt;mailto:<a href="mailto:Infra@ovirt.org">Infra@ovirt.org</a>&gt;<br>
&gt;     <a href="http://lists.ovirt.org/mailman/listinfo/infra" rel="noreferrer" target="_blank">http://lists.ovirt.org/<wbr>mailman/listinfo/infra</a><br>
<span class="im HOEnZb">&gt;     &lt;<a href="http://lists.ovirt.org/mailman/listinfo/infra" rel="noreferrer" target="_blank">http://lists.ovirt.org/<wbr>mailman/listinfo/infra</a>&gt;<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt; --<br>
&gt; Eyal Edri<br>
&gt; Associate Manager<br>
&gt; RHV DevOps<br>
&gt; EMEA ENG Virtualization R&amp;D<br>
&gt; Red Hat Israel<br>
&gt;<br>
&gt; phone: <a href="tel:%2B972-9-7692018" value="+97297692018">+972-9-7692018</a><br>
&gt; irc: eedri (on #tlv #rhev-dev #rhev-integ)<br>
<br>
<br>
</span><div class="HOEnZb"><div class="h5">--<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>
</div></div></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&amp;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>