<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, May 18, 2017 at 4:47 PM, Marcin Mirecki <span dir="ltr">&lt;<a href="mailto:mmirecki@redhat.com" target="_blank">mmirecki@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">The size of the required components:<br><div><br>Name   : openvswitch                      Size    : 11 M<br>Name   : openvswitch-ovn-common  Size    : 2.8 M<br>Name   : openvswitch-ovn-host        Size    : 1.9 M<br>Name   : ovirt-provider-ovn              Size    : 224 k<br>Name   : python-openvswitch          Size    : 821 k<br><br>about 17M total<br></div></div></blockquote><div><br></div><div>Latest nightly built appliance from master [1]  is 751MB so we are talking about a 2% increment.</div><div>If OVN if something that could interest a great part of our users, and personally I absolutely think so, I&#39;d suggest to simply include it. </div><div><br></div><div>[1] <a href="http://resources.ovirt.org/pub/ovirt-master-snapshot/rpm/el7/noarch/ovirt-engine-appliance-4.2-20170517.1.el7.centos.noarch.rpm">http://resources.ovirt.org/pub/ovirt-master-snapshot/rpm/el7/noarch/ovirt-engine-appliance-4.2-20170517.1.el7.centos.noarch.rpm</a><br></div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div></div></div><div class="gmail_extra"><div><div class="gmail-h5"><br><div class="gmail_quote">On Thu, May 18, 2017 at 3:26 PM, Simone Tiraboschi <span dir="ltr">&lt;<a href="mailto:stirabos@redhat.com" target="_blank">stirabos@redhat.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="gmail-m_-6293500916057615578h5">On Wed, May 17, 2017 at 2:35 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:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail-m_-6293500916057615578m_-2191800788320546677gmail-HOEnZb"><div class="gmail-m_-6293500916057615578m_-2191800788320546677gmail-h5">On Wed, May 17, 2017 at 2:54 PM, Dan Kenigsberg &lt;<a href="mailto:danken@redhat.com" target="_blank">danken@redhat.com</a>&gt; wrote:<br>
&gt; On Wed, May 17, 2017 at 12:23 PM, Yedidyah Bar David &lt;<a href="mailto:didi@redhat.com" target="_blank">didi@redhat.com</a>&gt; wrote:<br>
&gt;&gt; On Wed, May 17, 2017 at 11:42 AM, Dan Kenigsberg &lt;<a href="mailto:danken@redhat.com" target="_blank">danken@redhat.com</a>&gt; wrote:<br>
&gt;&gt;&gt; On Wed, May 17, 2017 at 10:44 AM, Yedidyah Bar David &lt;<a href="mailto:didi@redhat.com" target="_blank">didi@redhat.com</a>&gt; wrote:<br>
&gt;&gt;&gt;&gt; On Wed, May 17, 2017 at 9:38 AM, Dan Kenigsberg &lt;<a href="mailto:danken@redhat.com" target="_blank">danken@redhat.com</a>&gt; wrote:<br>
&gt;&gt;&gt;&gt;&gt; On Tue, May 16, 2017 at 6:56 PM, Sandro Bonazzola &lt;<a href="mailto:sbonazzo@redhat.com" target="_blank">sbonazzo@redhat.com</a>&gt; wrote:<br>
&gt;&gt;&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;&gt;&gt; Hi,<br>
&gt;&gt;&gt;&gt;&gt;&gt; with <a href="https://gerrit.ovirt.org/76855" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/76855</a> it&#39;s requested to increase the appliance size by adding ovirt-provider-ovn and its dependencies.<br>
&gt;&gt;&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;&gt;&gt; This raise a few questions.<br>
&gt;&gt;&gt;&gt;&gt;&gt; The support for ovirt-provider-ovn is enabled by default in engine-setup and going to be installed by default in the appliance so we&#39;re pushing to use it.<br>
&gt;&gt;&gt;&gt;&gt;&gt; Why not requiring it at ovirt-engine spec file level?<br>
&gt;&gt;&gt;&gt;&gt;&gt; Answer given in the commit message of above patch is:<br>
&gt;&gt;&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;&gt;&gt; We do not want to have a hard dependency in the<br>
&gt;&gt;&gt;&gt;&gt;&gt; form of an rpm require.<br>
&gt;&gt;&gt;&gt;&gt;&gt; OVN and openvswitch are relatively heavy and complex,<br>
&gt;&gt;&gt;&gt;&gt;&gt; and are still experimental. We would not want to<br>
&gt;&gt;&gt;&gt;&gt;&gt; force everybody to pull them onto any Engine host.<br>
&gt;&gt;&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;&gt;&gt; So why adding it to the appliance, which is the default for hosted engine which is our recommeded way to deploy oVirt, and enable it by default?<br>
&gt;&gt;&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;&gt;&gt; How this differs from DWH? ovirt-engine requires ovirt-engine-setup which requires ovirt-engine-dwh setup which requires ovirt-engine-dwh.<br>
&gt;&gt;&gt;&gt;&gt;&gt; Why can&#39;t we just require ovirt-provider-ovn in ovirt-engine instead of tweaking the appliance?<br>
&gt;&gt;&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;&gt;&gt; If we decide it&#39;s not mandatory, why not make the default to not enabling it in engine-setup and avoid to add it to the appliance?<br>
&gt;&gt;&gt;&gt;&gt;&gt; Being optional, adding it collides with Bug 1401931 - [RFE] reduce the size of the appliance<br>
&gt;&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;&gt; Much like with DWH, I can envisage a use case where ovirt-provider-ovn<br>
&gt;&gt;&gt;&gt;&gt; sits on a remote host, rather than on Engine&#39;s. However, the default<br>
&gt;&gt;&gt;&gt;&gt; use case is to place them on the same host.<br>
&gt;&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;&gt; I thought that it would be a good idea to include OVN on the<br>
&gt;&gt;&gt;&gt;&gt; appliance, as a means to showcase this new and exciting feature of<br>
&gt;&gt;&gt;&gt;&gt; oVirt. However, it is not a must. We can say that we&#39;d like to keep<br>
&gt;&gt;&gt;&gt;&gt; the appliance small; if someone wants to use OVN with it, let them run<br>
&gt;&gt;&gt;&gt;&gt; ovirt-engine-setup manually, and pull in the dependencies.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; The appliance is assumed to (soon?) be our standard installation flow,<br>
&gt;&gt;&gt;&gt; not a way to showcase things. For the latter, you might want to add ovn<br>
&gt;&gt;&gt;&gt; to ovirt-live or to the ovirt demo tool [1] (not yet released IIUC).<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; [1] <a href="https://trello.com/b/wocfflzf/sales-demo-tool-lago-based" rel="noreferrer" target="_blank">https://trello.com/b/wocfflzf/<wbr>sales-demo-tool-lago-based</a><br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt;&gt; For this we&#39;d need to flip the default, and not install OVN when the<br>
&gt;&gt;&gt;&gt;&gt; appliance is created, and skip OVN test in the offline test suite.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; +1<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; Could you point us to the answer file used for appliance creation?<br>
&gt;&gt;<br>
&gt;&gt; Do you want to keep the default True for non-appliance? My +1 above<br>
&gt;&gt; was also for reverting the default, not only in appliance.<br>
&gt;<br>
&gt; Oh. I still want to have OVN by default for non-appliance. I like this<br>
&gt; feature, and I want to entice people to use it.<br>
<br>
</div></div>I think that Sandro&#39;s question above applies equally well to the<br>
non-appliance usecase. If it&#39;s good enough to be the default for<br>
non-appliance, might as well be so for the appliance as well. If<br>
it&#39;s not good enough for the appliance, perhaps default to No also<br>
for non-appliance.<br>
<span class="gmail-m_-6293500916057615578m_-2191800788320546677gmail-"><br>
&gt;<br>
&gt; For appliance I understand that we have a size limitation, so ok, let<br>
&gt; us not bloat it up.<br>
<br>
</span>What&#39;s the impact on size? For the appliance image and for the<br>
eventually-installed machine?<br>
<br>
I do not think the impact on appliance size is the major question here,<br>
but whether we really expect most users to use OVN. But I might be<br>
surprised...<br>
<span class="gmail-m_-6293500916057615578m_-2191800788320546677gmail-im gmail-m_-6293500916057615578m_-2191800788320546677gmail-HOEnZb"><br></span></blockquote><div><br></div></div></div><div>Now we have a bug to track it:</div><div><a href="https://bugzilla.redhat.com/show_bug.cgi?id=1452131" target="_blank">https://bugzilla.redhat.com/sh<wbr>ow_bug.cgi?id=1452131</a><br></div><span><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-m_-6293500916057615578m_-2191800788320546677gmail-im gmail-m_-6293500916057615578m_-2191800788320546677gmail-HOEnZb">
&gt;<br>
&gt; I hope you are also fine with disabling ovn in the following answer file.<br>
&gt;<br>
&gt;&gt;<br>
&gt;&gt; The appliance-supplied answer file seems is:<br>
&gt;&gt;<br>
&gt;&gt; <a href="https://gerrit.ovirt.org/gitweb?p=ovirt-appliance.git;a=blob;f=engine-appliance/data/ovirt-engine-answers;h=2881af6563297a7a3d220dfe479d39f88c12ca46;hb=HEAD" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/gitwe<wbr>b?p=ovirt-appliance.git;a=blob<wbr>;f=engine-appliance/data/ovirt<wbr>-engine-answers;h=2881af656329<wbr>7a7a3d220dfe479d39f88c12ca46;<wbr>hb=HEAD</a><br>
&gt;&gt;<br>
&gt;&gt; When hosted-engine --deploy is using the appliance, and if the user<br>
&gt;&gt; asks to run engine-setup automatically, it uses above file,<br>
&gt;&gt; but also adds another file, auto-generated, see here:<br>
&gt;&gt;<br>
&gt;&gt; <a href="https://gerrit.ovirt.org/gitweb?p=ovirt-hosted-engine-setup.git;a=blob;f=src/plugins/gr-he-common/vm/cloud_init.py;h=0a20f946d65199423c99769ab51e4fe092465e96;hb=HEAD#l1018" rel="noreferrer" target="_blank">https://gerrit.ovirt.org/gitwe<wbr>b?p=ovirt-hosted-engine-setup.<wbr>git;a=blob;f=src/plugins/gr-he<wbr>-common/vm/cloud_init.py;h=0a2<wbr>0f946d65199423c99769ab51e4fe09<wbr>2465e96;hb=HEAD#l1018</a><br>
&gt;&gt;<br>
&gt;&gt; None of them has the answer for OVN. Latter has:<br>
&gt;&gt;<br>
&gt;&gt; DIALOG/autoAcceptDefault=bool:<wbr>True<br>
&gt;&gt;<br>
&gt;&gt; For this, see:<br>
&gt;&gt;<br>
&gt;&gt; <a href="https://bugzilla.redhat.com/show_bug.cgi?id=1270719" rel="noreferrer" target="_blank">https://bugzilla.redhat.com/sh<wbr>ow_bug.cgi?id=1270719</a><br>
<br>
<br>
<br>
</span><span class="gmail-m_-6293500916057615578m_-2191800788320546677gmail-HOEnZb"><font color="#888888">--<br>
Didi<br>
</font></span><div class="gmail-m_-6293500916057615578m_-2191800788320546677gmail-HOEnZb"><div class="gmail-m_-6293500916057615578m_-2191800788320546677gmail-h5">______________________________<wbr>_________________<br>
Devel mailing list<br>
<a href="mailto:Devel@ovirt.org" target="_blank">Devel@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ovirt.org/mailman<wbr>/listinfo/devel</a><br>
</div></div></blockquote></span></div><br></div></div>
<br>______________________________<wbr>_________________<br>
Devel mailing list<br>
<a href="mailto:Devel@ovirt.org" target="_blank">Devel@ovirt.org</a><br>
<a href="http://lists.ovirt.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ovirt.org/mailman<wbr>/listinfo/devel</a><br></blockquote></div><br><br clear="all"><br></div></div><span class="gmail-HOEnZb"><font color="#888888">-- <br><div class="gmail-m_-6293500916057615578gmail_signature"><div dir="ltr"><div>
<p style="font-weight:bold;margin:0px;padding:0px;font-size:14px;text-transform:uppercase">MARCIN mIRECKI<br></p>

<p style="font-weight:normal;margin:0px;font-size:10px;color:rgb(153,153,153)"><a style="color:rgb(0,136,206);font-size:10px;margin:0px;text-decoration:none;font-family:overpass,sans-serif" href="https://www.redhat.com" target="_blank">Red Hat <span><br><br></span></a></p>



<table border="0"><tbody><tr><td width="100px"><a href="https://red.ht/sig" target="_blank"> <img src="https://www.redhat.com/files/brand/email/sig-redhat.png" width="90" height="auto"></a> </td>
</tr></tbody></table>

</div></div></div>
</font></span></div>
</blockquote></div><br></div></div>