2:28 a.m.
On Thu, Nov 23, 2017 at 12:36 AM Barak Korren <bkorren(a)redhat.com> wrote:
On 22 November 2017 at 20:50, Nir Soffer <nsoffer(a)redhat.com>
wrote:
> Vdsm should be ready now for fc27 and fcraw builds:
> https://gerrit.ovirt.org/#/c/84368/
How was this verified?
Piotr, can you add more info about this?
> Can we enable the builds again?
I'd rather make sure we have proper verification so we don't end up
blocking the development again....
I'd would have been great if you'd let us temporarily enable the jobs
before merging that patch so we could let them run on it to verify.
I enabled the fc27 project temporarily, and it still fails when building
https://gerrit.ovirt.org/#/c/84368/ trying to import ovirt_imageio_common.
The package is specified in the packages file:
$ git grep ovirt-imageio-common automation/*.packages.fc27
automation/check-patch.packages.fc27:ovirt-imageio-common
We use these repos:
$ cat automation/check-patch.repos.fc27
tested,http://resources.ovirt.org/repos/ovirt/tested/master/rpm/$distro
virt-preview,
http://fedorapeople.org/groups/virt/virt-preview/fedora-$releasever/$base...
The package exists:
http://resources.ovirt.org/repos/ovirt/tested/master/rpm/fc27/noarch/ovir...
repoquery is happy:
$ repoquery --repofrompath=r,
http://resources.ovirt.org/repos/ovirt/tested/master/rpm/fc27 --repoid=r
list ovirt-imageio-common
ovirt-imageio-common-0:1.2.0-0.201711212128.git9926984.fc27.noarch
Maybe mock caching issue?
*00:00:28.127* Using chroot cache =
/var/cache/mock/fedora-27-x86_64-a9934b467f29c7317f7dd8f205d66ddd*00:00:28.127*
Using chroot dir =
/var/lib/mock/fedora-27-x86_64-a9934b467f29c7317f7dd8f205d66ddd-7425
http://jenkins.ovirt.org/job/vdsm_master_check-patch-fc27-x86_64/84/conso...
*00:01:59.428* =================================== FAILURES
===================================*00:01:59.429*
_________________________ test_import[vdsm.kvm2ovirt]
__________________________*00:01:59.430* *00:01:59.430* name =
'vdsm.kvm2ovirt'*00:01:59.430* *00:01:59.430*
@pytest.mark.parametrize("name", find_modules())*00:01:59.431* def
test_import(name):*00:01:59.431* try:*00:01:59.431* >
importlib.import_module(name)*00:01:59.432* *00:01:59.432*
check_imports.py:61: *00:01:59.432* _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ *00:01:59.433*
/usr/lib64/python2.7/importlib/__init__.py:37: in
import_module*00:01:59.434* __import__(name)*00:01:59.434* _ _ _ _
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _
_ *00:01:59.434* *00:01:59.435* from __future__ import
absolute_import*00:01:59.435* *00:01:59.435* import
argparse*00:01:59.435* from contextlib import
contextmanager*00:01:59.435* import itertools*00:01:59.435*
import sys*00:01:59.435* import threading*00:01:59.435*
*00:01:59.435* > from ovirt_imageio_common import
directio*00:01:59.435* E ImportError: No module named
ovirt_imageio_common*00:01:59.436* *00:01:59.436*
../lib/vdsm/kvm2ovirt.py:28: ImportError