<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 6, 2016 at 2:41 PM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Wed, Apr 6, 2016 at 2:33 PM, Edward Haas &lt;<a href="mailto:ehaas@redhat.com">ehaas@redhat.com</a>&gt; wrote:<br>
&gt;<br>
&gt;<br>
&gt; On Wed, Apr 6, 2016 at 2:23 PM, Nir Soffer &lt;<a href="mailto:nsoffer@redhat.com">nsoffer@redhat.com</a>&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt; On Wed, Apr 6, 2016 at 2:19 PM, Edward Haas &lt;<a href="mailto:ehaas@redhat.com">ehaas@redhat.com</a>&gt; wrote:<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; On Wed, Apr 6, 2016 at 1:41 PM, Milan Zamazal &lt;<a href="mailto:mzamazal@redhat.com">mzamazal@redhat.com</a>&gt;<br>
&gt;&gt; &gt; wrote:<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; Edward Haas &lt;<a href="mailto:ehaas@redhat.com">ehaas@redhat.com</a>&gt; writes:<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; &gt; On Wed, Apr 6, 2016 at 11:39 AM, Milan Zamazal &lt;<a href="mailto:mzamazal@redhat.com">mzamazal@redhat.com</a>&gt;<br>
&gt;&gt; &gt;&gt; &gt; wrote:<br>
&gt;&gt; &gt;&gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt;     Thank you, Edward, this is useful not only for CI. I use docker<br>
&gt;&gt; &gt;&gt; &gt; for<br>
&gt;&gt; &gt;&gt; &gt;     building Vdsm and running its unit tests and this helped me to<br>
&gt;&gt; &gt;&gt; &gt; get<br>
&gt;&gt; &gt;&gt; &gt; the<br>
&gt;&gt; &gt;&gt; &gt;     proper updated set of packages after recent changes in Vdsm.<br>
&gt;&gt; &gt;&gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt;     BTW, it seems that the following packages should be additionally<br>
&gt;&gt; &gt;&gt; &gt; added<br>
&gt;&gt; &gt;&gt; &gt;     for `make check-all&#39;: psmisc, which, python-ioprocess<br>
&gt;&gt; &gt;&gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt;<br>
&gt;&gt; &gt;&gt; &gt; Are you saying that make check is passing on your local machine?<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; When I add the packages given above, `make check-all&#39; (as well as `make<br>
&gt;&gt; &gt;&gt; check&#39;) works for me except for 4 tests in lib/vdsm/schedule.py that<br>
&gt;&gt; &gt;&gt; produce the following errors with `make check-all&#39;:<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt;     File &quot;/home/pdm/ovirt/vdsm/vdsm-test/lib/vdsm/schedule.py&quot;, line<br>
&gt;&gt; &gt;&gt; 134,<br>
&gt;&gt; &gt;&gt; in schedule<br>
&gt;&gt; &gt;&gt;       heapq.heappush(self._calls, (deadline, call))<br>
&gt;&gt; &gt;&gt;   nose.proxy.TypeError: unorderable types: ScheduledCall() &lt;<br>
&gt;&gt; &gt;&gt; ScheduledCall()<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt;     File &quot;/home/pdm/ovirt/vdsm/vdsm-test/tests/scheduleTests.py&quot;, line<br>
&gt;&gt; &gt;&gt; 160, in test_latency<br>
&gt;&gt; &gt;&gt;       med = ticker.latency[len(ticker.latency) / 2]<br>
&gt;&gt; &gt;&gt;   nose.proxy.TypeError: list indices must be integers, not float<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; Those are probably Python 3 failures that should be fixed in Vdsm.<br>
&gt;&gt; &gt;&gt; The docker environment works fine for running the unit tests on my<br>
&gt;&gt; &gt;&gt; machine.<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; I ran it on Travis CI with your recommended addition, and I am getting<br>
&gt;&gt; &gt; this<br>
&gt;&gt; &gt; result: FAILED (SKIP=107, errors=14):<br>
&gt;&gt; &gt; You can view the run here:<br>
&gt;&gt; &gt; <a href="https://travis-ci.org/EdDev/vdsm/builds/121117253" rel="noreferrer" target="_blank">https://travis-ci.org/EdDev/vdsm/builds/121117253</a><br>
&gt;&gt;<br>
&gt;&gt; Sure, make check in master run tests that should not run on travis.<br>
&gt;&gt;<br>
&gt;&gt; Try the travis branch - after adding ioprocess to the docker image,<br>
&gt;&gt; all tests should pass:<br>
&gt;&gt; <a href="https://gerrit.ovirt.org/55738" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/55738</a><br>
&gt;&gt;<br>
&gt;&gt; Nir<br>
&gt;<br>
&gt;<br>
&gt; Ok, will check it as well.<br>
&gt; But its a bit of a lie, many tests are skipped instead of not ran at all,<br>
&gt; this needs to be fixed.<br>
&gt; In addition, I find many tests as not unit tests, all tests should pass in a<br>
&gt; few seconds not in 2 minutes.<br>
<br>
</div></div>Storage people always lie :-)<br>
<br>
We have @slowtest for marking slow tests. Unfortunately, some tests are<br>
slow, and there is no value in mocking the thing you want to test.<br>
<span class="HOEnZb"><font color="#888888"><br>
Nir<br>
</font></span></blockquote></div><br></div><div class="gmail_extra">Well, I will like to run only the unit tests, and you seem to try and merge the integration tests<br></div><div class="gmail_extra">in the same run. I think they need to be separated.<br></div><div class="gmail_extra">I would like to see the unit tests pass in a few seconds, have no surprising skips and<br></div><div class="gmail_extra">have no dependency or other services.<br></div><div class="gmail_extra"><br></div></div>