<div dir="ltr">Hi All,<div><br></div><div>Yesterday we merged the class LibvirtVmXmlBuilder in ovirt-engine.</div><div>It does not include the conversion of all the previously supported VM properties (e.g., payload devices, OVS stuff, cinder disks - any help would be appreciated btw) or covers all flows (e.g., run once) yet and there is surely a place for improvement, but in its current state LibvirtVmXmlBuilder is able to generate a libvirt&#39;s domxml on the engine side for many typical VMs in our development environment.</div><div><br></div><div>If you happen to start working on a change that involves changing something in the domxml, please do it by modifying this class. Our plan is to deprecate the VM info map/dictionary soon.</div><div><br></div><div>Let me try to motivate you by example:</div><div><br></div><div>Let&#39;s say that you want to add support for a property of a disk called &#39;discard&#39;.</div><div>In 4.1:</div><div>You had to patch VmInfoBuilderImpl to add another element to the VM info map and then modify VDSM to use it when encoding the Disk device into XML.</div><div>Now:</div><div>You need to patch LibvirtVmXmlBuilder so the &#39;discard&#39; property would be set on the disk device element in the generated XML.</div><div><br></div><div>In order to test such a change you can:</div><div>(1) apply [1] to send the XML generated by the engine to VDSM</div><div>(2) apply [2] to send the XML generated by the engine to libvirt</div><div><br></div><div>[1] <a href="https://gerrit.ovirt.org/#/c/71088/">https://gerrit.ovirt.org/#/c/71088/</a></div><div>[2] <a href="https://gerrit.ovirt.org/#/c/65182/">https://gerrit.ovirt.org/#/c/65182/</a></div><div><br></div><div>Regards,</div><div>Arik</div></div>