<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Jun 20, 2016 at 4:47 PM, Markus Zoeller <span dir="ltr"><<a href="mailto:mzoeller@linux.vnet.ibm.com" target="_blank">mzoeller@linux.vnet.ibm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">White working on the change series to implement the virtlogd feature I<br>
got feedback [1] to move code which creates parts of the libvirt.xml<br>
file from the "driver" module into the "guest" module. I'm a bit<br>
hesitant to do so as the responsibility of creating a valid libvirt.xml<br>
file is then spread across 3 modules:<br>
* driver.py<br>
* guest.py<br>
* designer.py </blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I'm only looking for a guideline here (The "driver" module is humongous<br>
and I think it would be a good thing to have the "libvirt.xml" creation<br>
code outside of it). Thoughts?<br></blockquote><div><br></div><div>I'm also not a huge fan of the division of code for generating libvirt info different backends in imagebackend. I'm not familiar enough with the xml generation code to have a firm opinion on how it should be.</div><div><br></div><div>My concern looking at [1] is that to the greatest possible extent libvirt should be a data sink, not a data source. That is: Nova is the canonical source of truth. In terms of configuration, there should be nothing that libvirt knows that Nova didn't tell it first, so it should never be necessary to ask for it. The only reason this might not be true would be legacy instances, and we should be fixing them up.</div><div><br></div><div>So, eg, guest.get_path() shouldn't exist.</div><div> </div><div>Matt</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
References:<br>
[1]<br>
<a href="https://review.openstack.org/#/c/323761/2/nova/virt/libvirt/driver.py@4190" rel="noreferrer" target="_blank">https://review.openstack.org/#/c/323761/2/nova/virt/libvirt/driver.py@4190</a><br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Regards, Markus Zoeller (markus_z)<br>
<br>
<br>
__________________________________________________________________________<br>
OpenStack Development Mailing List (not for usage questions)<br>
Unsubscribe: <a href="http://OpenStack-dev-request@lists.openstack.org?subject:unsubscribe" rel="noreferrer" target="_blank">OpenStack-dev-request@lists.openstack.org?subject:unsubscribe</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" rel="noreferrer" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</font></span></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><span style="font-size:12.8px">Matthew Booth</span><br></div><div>Red Hat Engineering, Virtualisation Team</div><div><br></div><div>Phone: +442070094448 (UK)</div><div><br></div></div></div></div></div>
</div></div>