On Wed, Apr 6, 2016 at 2:23 PM, David Caro <dcaro@redhat.com> wrote:
On 04/06 14:19, Edward Haas wrote:
> On Wed, Apr 6, 2016 at 1:41 PM, Milan Zamazal <mzamazal@redhat.com> wrote:
>
> > Edward Haas <ehaas@redhat.com> writes:
> >
> > > On Wed, Apr 6, 2016 at 11:39 AM, Milan Zamazal <mzamazal@redhat.com>
> > 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


Afaik, you won't be able to run any tests that touch networking, or kernel
modules (bonding and such). That is as much a limitation of travis as of
docker, that was one of the points why we started using chroots instead of
docker containers on ovirt ci.

Yes, that is why I intended it for unit tests only, not for integration or functional.
We may find some integration tests that do pass, that needs some more investigation, but it's out of my original intent
.


> _______________________________________________
> Devel mailing list
> Devel@ovirt.org
> http://lists.ovirt.org/mailman/listinfo/devel


--
David Caro

Red Hat S.L.
Continuous Integration Engineer - EMEA ENG Virtualization R&D

Tel.: +420 532 294 605
Email: dcaro@redhat.com
IRC: dcaro|dcaroest@{freenode|oftc|redhat}
Web: www.redhat.com
RHT Global #: 82-62605