[VDSM] travis builds fails since "network: report dpdk devices as nics"

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

Sorry about the noise. Here is the fix: https://gerrit.ovirt.org/#/c/76740/ 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 <https://www.redhat.com/> <https://red.ht/sig> TRIED. TESTED. TRUSTED. <https://redhat.com/trusted> @redhatnews <https://twitter.com/redhatnews> Red Hat <https://www.linkedin.com/company/red-hat> Red Hat <https://www.facebook.com/RedHatInc>

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: https://hub.docker.com/u/ovirtorg/
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 <https://www.redhat.com/>
<https://red.ht/sig> TRIED. TESTED. TRUSTED. <https://redhat.com/trusted> @redhatnews <https://twitter.com/redhatnews> Red Hat <https://www.linkedin.com/company/red-hat> Red Hat <https://www.facebook.com/RedHatInc>

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: https://hub.docker.com/u/ovirtorg/
Merged, thanks!
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 <https://www.redhat.com/>
<https://red.ht/sig> TRIED. TESTED. TRUSTED. <https://redhat.com/trusted> @redhatnews <https://twitter.com/redhatnews> Red Hat <https://www.linkedin.com/company/red-hat> Red Hat <https://www.facebook.com/RedHatInc>

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: https://hub.docker.com/u/ovirtorg/
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 <https://www.redhat.com/>
<https://red.ht/sig> TRIED. TESTED. TRUSTED. <https://redhat.com/trusted> @redhatnews <https://twitter.com/redhatnews> Red Hat <https://www.linkedin.com/company/red-hat> Red Hat <https://www.facebook.com/RedHatInc>
participants (2)
-
Irit Goihman
-
Nir Soffer