On Thu, Nov 23, 2017 at 12:36 AM Barak Korren <bkorren@redhat.com> wrote:
On 22 November 2017 at 20:50, Nir Soffer <nsoffer@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/$basearc... The package exists: http://resources.ovirt.org/repos/ovirt/tested/master/rpm/fc27/noarch/ovirt-i... 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/consoleF... *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