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