On Wed, Apr 6, 2016 at 2:19 PM, Edward Haas <[email protected]> wrote: > > > On Wed, Apr 6, 2016 at 1:41 PM, Milan Zamazal <[email protected]> wrote: >> >> Edward Haas <[email protected]> writes: >> >> > On Wed, Apr 6, 2016 at 11:39 AM, Milan Zamazal <[email protected]> >> > wrote: >> > >> > Thank you, Edward, this is useful not only for CI. I use docker for >> > building Vdsm and running its unit tests and this helped me to get >> > the >> > proper updated set of packages after recent changes in Vdsm. >> > >> > BTW, it seems that the following packages should be additionally >> > added >> > for `make check-all': psmisc, which, python-ioprocess >> > >> > >> > Are you saying that make check is passing on your local machine? >> >> When I add the packages given above, `make check-all' (as well as `make >> check') works for me except for 4 tests in lib/vdsm/schedule.py that >> produce the following errors with `make check-all': >> >> File "/home/pdm/ovirt/vdsm/vdsm-test/lib/vdsm/schedule.py", line 134, >> in schedule >> heapq.heappush(self._calls, (deadline, call)) >> nose.proxy.TypeError: unorderable types: ScheduledCall() < >> ScheduledCall() >> >> File "/home/pdm/ovirt/vdsm/vdsm-test/tests/scheduleTests.py", line >> 160, in test_latency >> med = ticker.latency[len(ticker.latency) / 2] >> nose.proxy.TypeError: list indices must be integers, not float >> >> Those are probably Python 3 failures that should be fixed in Vdsm. >> The docker environment works fine for running the unit tests on my >> machine. > > > I ran it on Travis CI with your recommended addition, and I am getting this > result: FAILED (SKIP=107, errors=14): > You can view the run here: https://travis-ci.org/EdDev/vdsm/builds/121117253
Sure, make check in master run tests that should not run on travis. Try the travis branch - after adding ioprocess to the docker image, all tests should pass: https://gerrit.ovirt.org/55738 Nir _______________________________________________ Devel mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/devel
