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/$basearch

The package exists:

http://resources.ovirt.org/repos/ovirt/tested/master/rpm/fc27/noarch/ovirt-imageio-common-1.2.0-0.201711212128.git9926984.fc27.noarch.rpm

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/consoleFull

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