<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 2, 2017 at 12:30 PM, Yedidyah Bar David <span dir="ltr">&lt;<a href="mailto:didi@redhat.com" target="_blank">didi@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Thu, Nov 2, 2017 at 12:22 PM, Eyal Edri &lt;<a href="mailto:eedri@redhat.com">eedri@redhat.com</a>&gt; wrote:<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt; On Thu, Nov 2, 2017 at 12:12 PM, Yedidyah Bar David &lt;<a href="mailto:didi@redhat.com">didi@redhat.com</a>&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt; On Thu, Nov 2, 2017 at 9:56 AM, Sandro Bonazzola &lt;<a href="mailto:sbonazzo@redhat.com">sbonazzo@redhat.com</a>&gt; wrote:<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; 2017-11-02 8:46 GMT+01:00 Eyal Edri &lt;<a href="mailto:eedri@redhat.com">eedri@redhat.com</a>&gt;:<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; On Thu, Nov 2, 2017 at 9:24 AM, Sandro Bonazzola &lt;<a href="mailto:sbonazzo@redhat.com">sbonazzo@redhat.com</a>&gt; wrote:<br>
&gt;&gt; &gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt; 2017-11-01 17:16 GMT+01:00 Dafna Ron &lt;<a href="mailto:dron@redhat.com">dron@redhat.com</a>&gt;:<br>
&gt;&gt; &gt;&gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;&gt; Hi,<br>
&gt;&gt; &gt;&gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;&gt; we have failures in 4.1 imagebased.<br>
&gt;&gt; &gt;&gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;&gt; I think it&#39;s also related to the same package from 4.2 that is downloaded in the 4.1 repo.<br>
&gt;&gt; &gt;&gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;&gt; 2017-11-01 13:17:53,817::ssh.py::ssh::96:<wbr>:lago.ssh::DEBUG::Command 10946280 on lago-basic-suite-4-1-host-0  errors:<br>
&gt;&gt; &gt;&gt;&gt;&gt;  Error: Package: ovirt-host-4.2.0-0.0.master.<wbr>20171019135233.git1921fc6.el7.<wbr>centos.noarch (alocalsync)<br>
&gt;&gt; &gt;&gt;&gt;&gt;            Requires: vdsm-client &gt;= 4.20.0<br>
&gt;&gt; &gt;&gt;&gt;&gt;            Available: vdsm-client-4.19.35-2.<wbr>gitc1d5a55.el7.centos.noarch (alocalsync)<br>
&gt;&gt; &gt;&gt;&gt;&gt;                vdsm-client = 4.19.35-2.gitc1d5a55.el7.<wbr>centos<br>
&gt;&gt; &gt;&gt;&gt;&gt; Error: Package: ovirt-host-4.2.0-0.0.master.<wbr>20171019135233.git1921fc6.el7.<wbr>centos.noarch (alocalsync)<br>
&gt;&gt; &gt;&gt;&gt;&gt;            Requires: vdsm &gt;= 4.20.0<br>
&gt;&gt; &gt;&gt;&gt;&gt;            Available: vdsm-4.19.35-2.gitc1d5a55.el7.<wbr>centos.x86_64 (alocalsync)<br>
&gt;&gt; &gt;&gt;&gt;&gt;                vdsm = 4.19.35-2.gitc1d5a55.el7.<wbr>centos<br>
&gt;&gt;<br>
&gt;&gt; Where exactly is this failure? Please add links in such reports. Thanks.<br>
&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;&gt; From what I can see, the package is coming from the centos mirros:<br>
&gt;&gt; &gt;&gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;&gt; yum -y install ovirt-host<br>
&gt;&gt; &gt;&gt;&gt;&gt; rm -rf /dev/shm/*.rpm /dev/shm/yum<br>
&gt;&gt; &gt;&gt;&gt;&gt; &quot;<br>
&gt;&gt; &gt;&gt;&gt;&gt; 2017-11-01 13:17:53,817::ssh.py::ssh::81:<wbr>:lago.ssh::DEBUG::Command 10946280 on lago-basic-suite-4-1-host-0 returned with 0<br>
&gt;&gt; &gt;&gt;&gt;&gt; 2017-11-01 13:17:53,817::ssh.py::ssh::89:<wbr>:lago.ssh::DEBUG::Command 10946280 on lago-basic-suite-4-1-host-0 output:<br>
&gt;&gt; &gt;&gt;&gt;&gt;  Loaded plugins: fastestmirror<br>
&gt;&gt; &gt;&gt;&gt;&gt; Loading mirror speeds from cached hostfile<br>
&gt;&gt; &gt;&gt;&gt;&gt;  * base: <a href="http://mirror.keystealth.org" rel="noreferrer" target="_blank">mirror.keystealth.org</a><br>
&gt;&gt; &gt;&gt;&gt;&gt;  * extras: <a href="http://mirror.lax.hugeserver.com" rel="noreferrer" target="_blank">mirror.lax.hugeserver.com</a><br>
&gt;&gt; &gt;&gt;&gt;&gt;  * updates: <a href="http://dallas.tx.mirror.xygenhosting.com" rel="noreferrer" target="_blank">dallas.tx.mirror.xygenhosting.<wbr>com</a><br>
&gt;&gt; &gt;&gt;&gt;&gt; Resolving Dependencies<br>
&gt;&gt; &gt;&gt;&gt;&gt; --&gt; Running transaction check<br>
&gt;&gt; &gt;&gt;&gt;&gt; ---&gt; Package ovirt-host.noarch 0:4.2.0-0.0.master.<wbr>20171019135233.git1921fc6.el7.<wbr>centos will be installed<br>
&gt;&gt; &gt;&gt;&gt;&gt; --&gt; Processing Dependency: vdsm &gt;= 4.20.0 for package: ovirt-host-4.2.0-0.0.master.<wbr>20171019135233.git1921fc6.el7.<wbr>centos.noarch<br>
&gt;&gt; &gt;&gt;&gt;&gt; --&gt; Processing Dependency: vdsm-client &gt;= 4.20.0 for package: ovirt-host-4.2.0-0.0.master.<wbr>20171019135233.git1921fc6.el7.<wbr>centos.noarch<br>
&gt;&gt; &gt;&gt;&gt;&gt; --&gt; Processing Dependency: collectd for package: ovirt-host-4.2.0-0.0.master.<wbr>20171019135233.git1921fc6.el7.<wbr>centos.noarch<br>
&gt;&gt; &gt;&gt;&gt;&gt; --&gt; Processing Dependency: collectd-disk for package: ovirt-host-4.2.0-0.0.master.<wbr>20171019135233.git1921fc6.el7.<wbr>centos.noarch<br>
&gt;&gt; &gt;&gt;&gt;&gt; --&gt; Processing Dependency: collectd-netlink for package: ovirt-host-4.2.0-0.0.master.<wbr>20171019135233.git1921fc6.el7.<wbr>centos.noarch<br>
&gt;&gt; &gt;&gt;&gt;&gt; --&gt; Processing Dependency: collectd-virt for package: ovirt-host-4.2.0-0.0.master.<wbr>20171019135233.git1921fc6.el7.<wbr>centos.noarch<br>
&gt;&gt; &gt;&gt;&gt;&gt; --&gt; Processing Dependency: collectd-write_http for package: ovirt-host-4.2.0-0.0.master.<wbr>20171019135233.git1921fc6.el7.<wbr>centos.noarch<br>
&gt;&gt; &gt;&gt;&gt;&gt; --&gt; Processing Dependency: fluentd for package: ovirt-host-4.2.0-0.0.master.<wbr>20171019135233.git1921fc6.el7.<wbr>centos.noarch<br>
&gt;&gt; &gt;&gt;&gt;&gt; --&gt; Processing Dependency: ovirt-vmconsole for package: ovirt-host-4.2.0-0.0.master.<wbr>20171019135233.git1921fc6.el7.<wbr>centos.noarch<br>
&gt;&gt; &gt;&gt;&gt;&gt; --&gt; Processing Dependency: ovirt-vmconsole-host for package: ovirt-host-4.2.0-0.0.master.<wbr>20171019135233.git1921fc6.el7.<wbr>centos.noarch<br>
&gt;&gt; &gt;&gt;&gt;&gt; --&gt; Processing Dependency: rubygem-fluent-plugin-<wbr>collectd-nest for package: ovirt-host-4.2.0-0.0.master.<wbr>20171019135233.git1921fc6.el7.<wbr>centos.noarch<br>
&gt;&gt; &gt;&gt;&gt;&gt; --&gt; Processing Dependency: rubygem-fluent-plugin-rewrite-<wbr>tag-filter for package: ovirt-host-4.2.0-0.0.master.<wbr>20171019135233.git1921fc6.el7.<wbr>centos.noarch<br>
&gt;&gt; &gt;&gt;&gt;&gt; --&gt; Processing Dependency: rubygem-fluent-plugin-secure-<wbr>forward for package: ovirt-host-4.2.0-0.0.master.<wbr>20171019135233.git1921fc6.el7.<wbr>centos.noarch<br>
&gt;&gt; &gt;&gt;&gt;&gt; --&gt; Processing Dependency: rubygem-fluent-plugin-viaq_<wbr>data_model for package: ovirt-host-4.2.0-0.0.master.<wbr>20171019135233.git1921fc6.el7.<wbr>centos.noarch<br>
&gt;&gt; &gt;&gt;&gt;&gt; --&gt; Processing Dependency: socat for package: <a href="http://ovirt-host-4.2.0-0.0.master.20171019135233.git1921fc6.el7.centos.no" rel="noreferrer" target="_blank">ovirt-host-4.2.0-0.0.master.<wbr>20171019135233.git1921fc6.el7.<wbr>centos.no</a><br>
&gt;&gt; &gt;&gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;&gt; Sandro, can you please help to remove this package?<br>
&gt;&gt; &gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt; Removed from tested repo. It should have been ovirt-host-4.1.0 not 4.2.0. Not sure how it got there passing CI.<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; It got there before the test that was trying to install it was added to OST, so nothing prevented it from being pushed to 4.2.<br>
&gt;&gt;<br>
&gt;&gt; Not sure I fully understand. What test does install it now?<br>
&gt;&gt; All I see is [1], which is a performance cheat/hack, not a proper test,<br>
&gt;&gt; and was merged long ago (1.5 weeks before I started breaking ovirt-host-4.1).<br>
&gt;&gt;<br>
&gt;&gt; In principle, only the engine should instruct to install ovirt-host<br>
&gt;&gt; during *testing*, and only a 4.2 engine. Do we have tests that add<br>
&gt;&gt; a 4.1 host to a 4.2 engine?<br>
&gt;<br>
&gt;<br>
&gt; This was another bug in OST, the installation of ovirt-host should only have be done on 4.2, its also fixed now so future runs should install ovirt-host only on 4.2.<br>
<br>
</div></div>If in &quot;another bug&quot; you mean &quot;the reason for [1]&quot; (it has no bug-url),<br></blockquote><div><br></div><div>Yea this is what I meant, maybe &#39;bug&#39; was a wrong word for it, the author didn&#39;t intended to have ovirt-host</div><div>test added to 4.1 at this point, but it got in due to shared LagoInitFile.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
then I do not think it&#39;s a bug, at least not now (after 4.1 repo is fixed).<br>
Installing ovirt-host-4.1 for a 4.1 host should have exactly the same<br>
principle effect as installing ovirt-host-4.2 for a 4.2 host. The only<br>
difference is that only 4.2 engines /rely/ on this behavior, but if<br>
the patch before [1] was only added as a performance hack, there is no<br>
problem in using it also for 4.1.<br></blockquote><div><br></div><div>If you think its valuable and relevant, please send a patch to add it to 4.1, I got the feeling we only want to test it on 4.2.</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
[1] <a href="https://gerrit.ovirt.org/83448" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/83448</a><br>
<div><div class="h5"><br>
&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; [1] <a href="https://gerrit.ovirt.org/82461" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/82461</a><br>
&gt;&gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; ok, this explain<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; If you know that a certain pkg got built with the wrong version, please let us know so we can act on it and perhaps avoid such failures.<br>
&gt;&gt;<br>
&gt;&gt; Sorry, that&#39;s my fault, and I didn&#39;t think about it.<br>
&gt;&gt; As Sandro wrote in private, I started the work on 4.1<br>
&gt;&gt; by cherry-picking patches for master that I decided we<br>
&gt;&gt; need in 4.1, and only later changed the version to 4.1.<br>
&gt;&gt; In retrospective, I should probably have done this adaptation<br>
&gt;&gt; as the first patch in the series.<br>
&gt;&gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; I&#39;ve been made aware this morning<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; The 4.1 build was changed in [1] but before that it was building 4.2 version.<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; Noted that while investigating this morning<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; [1] <a href="https://gerrit.ovirt.org/#/c/82875/" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/#/c/<wbr>82875/</a><br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;&gt; Thanks,<br>
&gt;&gt; &gt;&gt;&gt;&gt; Dafna<br>
&gt;&gt; &gt;&gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt; --<br>
&gt;&gt; &gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt; SANDRO BONAZZOLA<br>
&gt;&gt; &gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt; ASSOCIATE MANAGER, SOFTWARE ENGINEERING, EMEA ENG VIRTUALIZATION R&amp;D<br>
&gt;&gt; &gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt; Red Hat EMEA<br>
&gt;&gt; &gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt; TRIED. TESTED. TRUSTED.<br>
&gt;&gt; &gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt;<br>
&gt;&gt; &gt;&gt;&gt; ______________________________<wbr>_________________<br>
&gt;&gt; &gt;&gt;&gt; Devel mailing list<br>
&gt;&gt; &gt;&gt;&gt; <a href="mailto:Devel@ovirt.org">Devel@ovirt.org</a><br>
&gt;&gt; &gt;&gt;&gt; <a href="http://lists.ovirt.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ovirt.org/<wbr>mailman/listinfo/devel</a><br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; --<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; Eyal edri<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; MANAGER<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; RHV DevOps<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; EMEA VIRTUALIZATION R&amp;D<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; Red Hat EMEA<br>
&gt;&gt; &gt;&gt;<br>
&gt;&gt; &gt;&gt; TRIED. TESTED. TRUSTED.<br>
&gt;&gt; &gt;&gt; phone: <a href="tel:%2B972-9-7692018" value="+97297692018">+972-9-7692018</a><br>
&gt;&gt; &gt;&gt; irc: eedri (on #tlv #rhev-dev #rhev-integ)<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; --<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; SANDRO BONAZZOLA<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; ASSOCIATE MANAGER, SOFTWARE ENGINEERING, EMEA ENG VIRTUALIZATION R&amp;D<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; Red Hat EMEA<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; TRIED. TESTED. TRUSTED.<br>
&gt;&gt; &gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; --<br>
&gt;&gt; Didi<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt;<br>
&gt; --<br>
&gt;<br>
&gt; Eyal edri<br>
&gt;<br>
&gt;<br>
&gt; MANAGER<br>
&gt;<br>
&gt; RHV DevOps<br>
&gt;<br>
&gt; EMEA VIRTUALIZATION R&amp;D<br>
&gt;<br>
&gt;<br>
&gt; Red Hat EMEA<br>
&gt;<br>
&gt; TRIED. TESTED. TRUSTED.<br>
&gt; phone: <a href="tel:%2B972-9-7692018" value="+97297692018">+972-9-7692018</a><br>
&gt; irc: eedri (on #tlv #rhev-dev #rhev-integ)<br>
<br>
<br>
<br>
<br>
--<br>
</div></div>Didi<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><p style="font-family:overpass,sans-serif;margin:0px;padding:0px;font-size:14px;text-transform:uppercase;font-weight:bold"><font color="#cc0000">Eyal edri</font></p><p style="color:rgb(0,0,0);font-family:overpass,sans-serif;font-weight:bold;margin:0px;padding:0px;font-size:14px;text-transform:uppercase"><br></p><p style="color:rgb(0,0,0);font-family:overpass,sans-serif;font-size:10px;margin:0px 0px 4px;text-transform:uppercase">MANAGER</p><p style="color:rgb(0,0,0);font-family:overpass,sans-serif;font-size:10px;margin:0px 0px 4px;text-transform:uppercase">RHV DevOps</p><p style="color:rgb(0,0,0);font-family:overpass,sans-serif;font-size:10px;margin:0px 0px 4px;text-transform:uppercase">EMEA VIRTUALIZATION R&amp;D</p><p style="color:rgb(0,0,0);font-family:overpass,sans-serif;font-size:10px;margin:0px 0px 4px;text-transform:uppercase"><br></p><p style="font-family:overpass,sans-serif;margin:0px;font-size:10px;color:rgb(153,153,153)"><a href="https://www.redhat.com/" style="color:rgb(0,136,206);margin:0px" target="_blank">Red Hat EMEA</a></p><table border="0" style="color:rgb(0,0,0);font-family:overpass,sans-serif;font-size:medium"><tbody><tr><td width="100px"><a href="https://red.ht/sig" style="color:rgb(17,85,204)" target="_blank"><img src="https://www.redhat.com/profiles/rh/themes/redhatdotcom/img/logo-red-hat-black.png" width="90" height="auto"></a></td><td style="font-size:10px"><a href="https://redhat.com/trusted" style="color:rgb(204,0,0);font-weight:bold" target="_blank">TRIED. TESTED. TRUSTED.</a></td></tr></tbody></table></div><div>phone: +972-9-7692018<br>irc: eedri (on #tlv #rhev-dev #rhev-integ)</div></div></div></div></div></div></div></div></div></div></div>
</div></div>