<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Thu, Nov 23, 2017 at 4:55 PM Dan Kenigsberg &lt;<a href="mailto:danken@redhat.com">danken@redhat.com</a>&gt; wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Nov 23, 2017 at 4:03 PM, Nir Soffer &lt;<a href="mailto:nsoffer@redhat.com" target="_blank">nsoffer@redhat.com</a>&gt; wrote:<br>
&gt; On Thu, Nov 23, 2017 at 3:59 PM Dan Kenigsberg &lt;<a href="mailto:danken@redhat.com" target="_blank">danken@redhat.com</a>&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt; On Thu, Nov 23, 2017 at 1:56 PM, Nir Soffer &lt;<a href="mailto:nsoffer@redhat.com" target="_blank">nsoffer@redhat.com</a>&gt; wrote:<br>
&gt;&gt; &gt; On Thu, Nov 23, 2017 at 1:51 PM Edward Haas &lt;<a href="mailto:ehaas@redhat.com" target="_blank">ehaas@redhat.com</a>&gt; wrote:<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; Per what I see, all CI jobs on vdsm/fc27 fail.<br>
&gt;&gt; &gt;&gt; This is the second time this week, please consider reverting.<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; We should try to avoid such changed before the weekend<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; Some of the failures like<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; <a href="http://jenkins.ovirt.org/job/vdsm_master_check-patch-fc27-x86_64/100/console" rel="noreferrer" target="_blank">http://jenkins.ovirt.org/job/vdsm_master_check-patch-fc27-x86_64/100/console</a><br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; Should be fixed by:<br>
&gt;&gt; &gt; <a href="https://gerrit.ovirt.org/84569/" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/84569/</a><br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; I have see some network test failing when running mock_runner locally,<br>
&gt;&gt; &gt; you may need to mark them as broken on fc27.<br>
&gt;&gt;<br>
&gt;&gt; We have a busy couple of weeks until release of ovirt-4.2.0. As much<br>
&gt;&gt; as I like consuming Fedora early, I&#39;m not sure that enabling so close<br>
&gt;&gt; to the release was a good idea. Nothing forces us to do it now (and<br>
&gt;&gt; there are a lot of reasons to do it later). Let&#39;s give it another go,<br>
&gt;&gt; but let us not keep it on the red for the weekend.<br>
&gt;<br>
&gt;<br>
&gt; I&#39;m happy with the storage code being tested on current fedora.<br>
<br>
I&#39;m more than happy. I&#39;m thrilled to for it to be tested and run.<br>
I am not happy to give a vote to a job that was never ever successful.<br></blockquote><div><br></div><div>The job is successful, some tests or maybe the code they test need work.</div><div>This is why we have skipif/xfail and broken_on_ci.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
&gt;<br>
&gt; Please make sure the few failing network tests are not breaking the build.<br>
<br>
It&#39;s probably time to mark the tests that use loseup as broken-on-jenkins<br></blockquote><div><br></div><div>loop devices are usually ok on jenkins. We have several tests in storage using</div><div>them and I don&#39;t know about any failures. For example storage/blockdev_test.py.</div><div><br></div><div>Francesco, do you want to mark them as broken for now?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
ERROR: Tests mkimage.injectFilesToFs creating an image and checking its<br>
----------------------------------------------------------------------<br>
Traceback (most recent call last):<br>
  File &quot;/home/jenkins/workspace/vdsm_master_check-patch-el7-x86_64/vdsm/tests/testlib.py&quot;,<br>
line 143, in wrapper<br>
    return f(self, *args)<br>
  File &quot;/home/jenkins/workspace/vdsm_master_check-patch-el7-x86_64/vdsm/tests/testValidation.py&quot;,<br>
line 191, in wrapper<br>
    return f(*args, **kwargs)<br>
  File &quot;/home/jenkins/workspace/vdsm_master_check-patch-el7-x86_64/vdsm/tests/mkimage_test.py&quot;,<br>
line 176, in test_injectFilesToFs<br>
    mkimage.injectFilesToFs(floppy, self.files, fstype)<br>
  File &quot;/home/jenkins/workspace/vdsm_master_check-patch-el7-x86_64/vdsm/lib/vdsm/mkimage.py&quot;,<br>
line 112, in injectFilesToFs<br>
    m.mount(mntOpts=&#39;loop&#39;, vfstype=fstype)<br>
  File &quot;/home/jenkins/workspace/vdsm_master_check-patch-el7-x86_64/vdsm/lib/vdsm/storage/mount.py&quot;,<br>
line 208, in mount<br>
    cgroup=cgroup)<br>
  File &quot;/home/jenkins/workspace/vdsm_master_check-patch-el7-x86_64/vdsm/lib/vdsm/storage/mount.py&quot;,<br>
line 278, in _mount<br>
    _runcmd(cmd)<br>
  File &quot;/home/jenkins/workspace/vdsm_master_check-patch-el7-x86_64/vdsm/lib/vdsm/storage/mount.py&quot;,<br>
line 306, in _runcmd<br>
    raise MountError(rc, &quot;;&quot;.join((out, err)))<br>
MountError: (32, &#39;;mount:<br>
/tmp/vdsm-mkimage-testspahtvm/images/vmId_inject.7301b08f8c0dc496bedd25577dbddbd8.img:<br>
failed to setup loop device: No such file or directory\n&#39;)<br>
<br>
<br>
<br>
&gt;<br>
&gt; Nir<br>
&gt;<br>
&gt;<br>
</blockquote></div></div>