On Thu, May 11, 2017 at 8:33 PM Nir Soffer <nsoffer@redhat.com> wrote:
On Thu, May 11, 2017 at 8:12 PM Nir Soffer <nsoffer@redhat.com> wrote:
On Thu, May 11, 2017 at 7:44 PM Irit Goihman <igoihman@redhat.com> wrote:
Sorry about the noise.
Here is the fix: https://gerrit.ovirt.org/#/c/76740/

Thanks, you can check the docker builds here:

Merged, thanks!

We are almost done, now only the fedora 25 build fail with:

    AttributeError: /vdsm/.tox/imports/bin/python: undefined symbol: glfs_new 

Patch https://gerrit.ovirt.org/76743 fix error handling so we see the real error:

    OSError: libgfapi.so.0: cannot open shared object file: No such file or directory

And patch https://gerrit.ovirt.org/76744 adds the missing glusterfs-api package.

With both patches travis builds are green again:
https://travis-ci.org/oVirt/vdsm/builds

Please review.

Nir

 
 

On Thu, May 11, 2017 at 6:49 PM, Nir Soffer <nsoffer@redhat.com> wrote:
Travis builds are failing because the lshw package was not added
to the vdsm test docker images.

The offending patch is:

network: report dpdk devices as nics
added link_info dictionary for dpdk dev information.
Currently it contains hard coded and empty values.

getLinks now reports dpdk devices as nics and providing
their link_info as well.

Signed-off-by: Irit Goihman <igoihman@redhat.com>
Change-Id: I67b2083e9c6f032472fd75f6800c864083ffdef5

When adding new build dependencies, please add the
required packages to docker/Dockerfile.* 

Thanks,
Nir




--

IRIT GOIHMAN

SOFTWARE ENGINEER

EMEA VIRTUALIZATION R&D

Red Hat EMEA