<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 14, 2016 at 10:59 AM, Nir Soffer <span dir="ltr">&lt;<a href="mailto:nsoffer@redhat.com" target="_blank">nsoffer@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><span>On Thu, Apr 14, 2016 at 10:45 AM, Yaniv Bronheim &lt;<a href="mailto:ybronhei@redhat.com" target="_blank">ybronhei@redhat.com</a>&gt; wrote:<br>
</span><span>&gt; I don&#39;t think this package is available in epel. if not, just remove it from<br>
&gt; py3 list<br>
&gt;<br>
&gt; about why we didn&#39;t catch it in jenkins - its because I don&#39;t run &quot;make<br>
&gt; check&quot; anymore over el7 to save resources. we just build the rpm there to<br>
&gt; see that we don&#39;t miss any dependencies.<br>
&gt; maybe we should bring back the make check there ... what do you think?<br>
<br>
</span>We should, make check takes about 1.5 minutes, typical build time is<br>
about 10 minutes<br>
<br>
fc23 build, with make check: 10:19<br>
<a href="http://jenkins.ovirt.org/job/vdsm_master_check-patch-fc23-x86_64/5210/console" rel="noreferrer" target="_blank">http://jenkins.ovirt.org/job/vdsm_master_check-patch-fc23-x86_64/5210/console</a></blockquote><div><br></div><div>It can be drastically reduced, just by running it into RAM. </div><div>I can run both EL7 and FC23 concurrently, on my laptop, @ about 5:15. Till they fail.</div><div><br></div><div>EL7 make check fails on:<br></div><div><div>/usr/bin/pep8 --exclude=&quot;${exclude}&quot; --filename &#39;*.py&#39; . \</div><div>contrib/logdb contrib/profile-stats contrib/repoplot init/daemonAdapter vdsm/get-conf-item vdsm/set-conf-item vdsm/supervdsmServer vdsm/vdsm vdsm/vdsm-restore-net-config vdsm/storage/curl-img-wrap vdsm/storage/fc-scan vdsm-tool/vdsm-tool</div><div>./vdsm_hooks/checkips/after_get_stats.py:81:17: E126 continuation line over-indented for hanging indent</div><div>./vdsm_hooks/checkips/after_get_stats.py:88:21: E126 continuation line over-indented for hanging indent</div><div>./vdsm_hooks/checkips/after_get_stats.py:94:21: E126 continuation line over-indented for hanging indent</div><div>make: *** [pep8] Error 1</div></div><div><br></div><div><br></div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><br>
<br>
el7 build, no make check: 11:53<br>
<a href="http://jenkins.ovirt.org/job/vdsm_master_check-patch-el7-x86_64/747/console" rel="noreferrer" target="_blank">http://jenkins.ovirt.org/job/vdsm_master_check-patch-el7-x86_64/747/console</a></blockquote><div><br></div><div>I&#39;m running both EL7 and FC23 in parallel.</div><div>Y.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><br>
<br>
travis build: 3-4 minutes<br>
<a href="https://travis-ci.org/nirs/vdsm/builds" rel="noreferrer" target="_blank">https://travis-ci.org/nirs/vdsm/builds</a><br>
<br>
The time of the tests does not make a real difference, and we know<br>
that the (some) code<br>
works on both platforms.<br>
<br>
We should work on reducing the build time, 10 minutes for running<br>
tests that take<br>
1.5 minutes is crazy overhead.<br>
<span><font color="#888888"><br>
Nir<br>
</font></span><div><div><br>
&gt;<br>
&gt; On Thu, Apr 14, 2016 at 10:26 AM, Francesco Romani &lt;<a href="mailto:fromani@redhat.com" target="_blank">fromani@redhat.com</a>&gt;<br>
&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; ________________________________<br>
&gt;&gt;<br>
&gt;&gt; From: &quot;Sandro Bonazzola&quot; &lt;<a href="mailto:sbonazzo@redhat.com" target="_blank">sbonazzo@redhat.com</a>&gt;<br>
&gt;&gt; To: &quot;Francesco Romani&quot; &lt;<a href="mailto:fromani@redhat.com" target="_blank">fromani@redhat.com</a>&gt;<br>
&gt;&gt; Cc: &quot;Eyal Edri&quot; &lt;<a href="mailto:eedri@redhat.com" target="_blank">eedri@redhat.com</a>&gt;, &quot;Dan Kenigsberg&quot; &lt;<a href="mailto:danken@redhat.com" target="_blank">danken@redhat.com</a>&gt;,<br>
&gt;&gt; &quot;devel&quot; &lt;<a href="mailto:devel@ovirt.org" target="_blank">devel@ovirt.org</a>&gt;, &quot;Yaniv Bronheim&quot; &lt;<a href="mailto:ybronhei@redhat.com" target="_blank">ybronhei@redhat.com</a>&gt;, &quot;Nir<br>
&gt;&gt; Soffer&quot; &lt;<a href="mailto:nsoffer@redhat.com" target="_blank">nsoffer@redhat.com</a>&gt;<br>
&gt;&gt; Sent: Thursday, April 14, 2016 9:13:04 AM<br>
&gt;&gt;<br>
&gt;&gt; Subject: Re: [ovirt-devel] [ACTION REQUIRED]<br>
&gt;&gt; vdsm_master_build-artifacts-fc23-x86_64 is failing due to missing dep on<br>
&gt;&gt; fc23 .packages<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; On Thu, Apr 14, 2016 at 9:12 AM, Sandro Bonazzola &lt;<a href="mailto:sbonazzo@redhat.com" target="_blank">sbonazzo@redhat.com</a>&gt;<br>
&gt;&gt; wrote:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; On Thu, Apr 14, 2016 at 9:01 AM, Francesco Romani &lt;<a href="mailto:fromani@redhat.com" target="_blank">fromani@redhat.com</a>&gt;<br>
&gt;&gt;&gt; wrote:<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; ________________________________<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; From: &quot;Eyal Edri&quot; &lt;<a href="mailto:eedri@redhat.com" target="_blank">eedri@redhat.com</a>&gt;<br>
&gt;&gt;&gt;&gt; To: &quot;Sandro Bonazzola&quot; &lt;<a href="mailto:sbonazzo@redhat.com" target="_blank">sbonazzo@redhat.com</a>&gt;<br>
&gt;&gt;&gt;&gt; Cc: &quot;Dan Kenigsberg&quot; &lt;<a href="mailto:danken@redhat.com" target="_blank">danken@redhat.com</a>&gt;, &quot;devel&quot; &lt;<a href="mailto:devel@ovirt.org" target="_blank">devel@ovirt.org</a>&gt;,<br>
&gt;&gt;&gt;&gt; &quot;Yaniv Bronheim&quot; &lt;<a href="mailto:ybronhei@redhat.com" target="_blank">ybronhei@redhat.com</a>&gt;, &quot;Nir Soffer&quot; &lt;<a href="mailto:nsoffer@redhat.com" target="_blank">nsoffer@redhat.com</a>&gt;,<br>
&gt;&gt;&gt;&gt; &quot;Francesco Romani&quot; &lt;<a href="mailto:fromani@redhat.com" target="_blank">fromani@redhat.com</a>&gt;<br>
&gt;&gt;&gt;&gt; Sent: Thursday, April 14, 2016 8:54:50 AM<br>
&gt;&gt;&gt;&gt; Subject: Re: [ovirt-devel] [ACTION REQUIRED]<br>
&gt;&gt;&gt;&gt; vdsm_master_build-artifacts-fc23-x86_64 is failing due to missing dep on<br>
&gt;&gt;&gt;&gt; fc23 .packages<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; Don&#39;t we run it per patch as well?<br>
&gt;&gt;&gt;&gt; How did it got merged?<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; On Apr 14, 2016 9:42 AM, &quot;Sandro Bonazzola&quot; &lt;<a href="mailto:sbonazzo@redhat.com" target="_blank">sbonazzo@redhat.com</a>&gt; wrote:<br>
&gt;&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;&gt; <a href="http://jenkins.ovirt.org/job/vdsm_master_build-artifacts-fc23-x86_64/823/console" rel="noreferrer" target="_blank">http://jenkins.ovirt.org/job/vdsm_master_build-artifacts-fc23-x86_64/823/console</a><br>
&gt;&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.751<br>
&gt;&gt;&gt;&gt;&gt; ======================================================================<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.751 ERROR: Failure: ImportError (No module named &#39;netaddr&#39;)<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.751<br>
&gt;&gt;&gt;&gt;&gt; ----------------------------------------------------------------------<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.752 Traceback (most recent call last):<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.752   File &quot;/usr/lib/python3.4/site-packages/nose/failure.py&quot;,<br>
&gt;&gt;&gt;&gt;&gt; line 39, in runTest<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.752     raise self.exc_val.with_traceback(self.tb)<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.752   File &quot;/usr/lib/python3.4/site-packages/nose/loader.py&quot;,<br>
&gt;&gt;&gt;&gt;&gt; line 418, in loadTestsFromName<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.752     addr.filename, addr.module)<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.752   File<br>
&gt;&gt;&gt;&gt;&gt; &quot;/usr/lib/python3.4/site-packages/nose/importer.py&quot;, line 47, in<br>
&gt;&gt;&gt;&gt;&gt; importFromPath<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.752     return self.importFromDir(dir_path, fqname)<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.752   File<br>
&gt;&gt;&gt;&gt;&gt; &quot;/usr/lib/python3.4/site-packages/nose/importer.py&quot;, line 94, in<br>
&gt;&gt;&gt;&gt;&gt; importFromDir<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.753     mod = load_module(part_fqname, fh, filename, desc)<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.753   File &quot;/usr/lib64/python3.4/imp.py&quot;, line 235, in<br>
&gt;&gt;&gt;&gt;&gt; load_module<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.753     return load_source(name, filename, file)<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.753   File &quot;/usr/lib64/python3.4/imp.py&quot;, line 171, in<br>
&gt;&gt;&gt;&gt;&gt; load_source<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.753     module = methods.load()<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.753   File &quot;&lt;frozen importlib._bootstrap&gt;&quot;, line 1220, in load<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.753   File &quot;&lt;frozen importlib._bootstrap&gt;&quot;, line 1200, in<br>
&gt;&gt;&gt;&gt;&gt; _load_unlocked<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.753   File &quot;&lt;frozen importlib._bootstrap&gt;&quot;, line 1129, in<br>
&gt;&gt;&gt;&gt;&gt; _exec<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.753   File &quot;&lt;frozen importlib._bootstrap&gt;&quot;, line 1471, in<br>
&gt;&gt;&gt;&gt;&gt; exec_module<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.754   File &quot;&lt;frozen importlib._bootstrap&gt;&quot;, line 321, in<br>
&gt;&gt;&gt;&gt;&gt; _call_with_frames_removed<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.754   File<br>
&gt;&gt;&gt;&gt;&gt; &quot;/home/jenkins/workspace/vdsm_master_build-artifacts-fc23-x86_64/vdsm/rpmbuild/BUILD/vdsm-4.17.999/tests/network/models_test.py&quot;,<br>
&gt;&gt;&gt;&gt;&gt; line 27, in &lt;module&gt;<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.754     from vdsm.netinfo import bonding, mtus<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.754   File<br>
&gt;&gt;&gt;&gt;&gt; &quot;/home/jenkins/workspace/vdsm_master_build-artifacts-fc23-x86_64/vdsm/rpmbuild/BUILD/vdsm-4.17.999/lib/vdsm/netinfo/__init__.py&quot;,<br>
&gt;&gt;&gt;&gt;&gt; line 26, in &lt;module&gt;<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.754     from .ipwrapper import DUMMY_BRIDGE<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.755   File<br>
&gt;&gt;&gt;&gt;&gt; &quot;/home/jenkins/workspace/vdsm_master_build-artifacts-fc23-x86_64/vdsm/rpmbuild/BUILD/vdsm-4.17.999/lib/vdsm/ipwrapper.py&quot;,<br>
&gt;&gt;&gt;&gt;&gt; line 30, in &lt;module&gt;<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.755     from netaddr.core import AddrFormatError<br>
&gt;&gt;&gt;&gt;&gt; 00:05:46.755 ImportError: No module named &#39;netaddr&#39;<br>
&gt;&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;&gt; Please fix ASAP.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; I think this a jenkins failure, because this seems to come from (lack<br>
&gt;&gt;&gt;&gt; of) netaddr package, which is an external package Vdsm depends on since ages<br>
&gt;&gt;&gt;&gt; (late 2013!)<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; I think it&#39;s a missing dep in build-artifacts.packages<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; forgot to mention, please note it&#39;s the python3 package missing.<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; yep, I think you are right. Most likely broken by<br>
&gt;&gt; <a href="https://gerrit.ovirt.org/#/c/55896/" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/55896/</a><br>
&gt;&gt;<br>
&gt;&gt; Let me quickly post a patch for this.<br>
&gt;&gt;<br>
&gt;&gt; Bests,<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; --<br>
&gt;&gt; Francesco Romani<br>
&gt;&gt; RedHat Engineering Virtualization R &amp; D<br>
&gt;&gt; Phone: 8261328<br>
&gt;&gt; IRC: fromani<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt; --<br>
&gt; Yaniv Bronhaim.<br>
</div></div><div><div>_______________________________________________<br>
Devel mailing list<br>
<a href="mailto:Devel@ovirt.org" target="_blank">Devel@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ovirt.org/mailman/listinfo/devel</a><br>
</div></div></blockquote></div><br></div></div>