<div dir="ltr"><div>Hi,</div><div>we have found an issue caused by 321a28c6 "libxl: set default disk format in device post-parse" (and maybe more changes I couldn't identify yet).</div><div><br></div><div>TL;DR: The auto-added driver section in xen changed and now causes issues.</div><div><br></div><div>An example to to trigger:</div><div>1. Create a  XEN xml with a disk device of type cdrom, but do not add a driver section, like.</div><div>    <disk type='file' device='cdrom'></div><div>      <target dev='hdb' bus='ide'/></div><div>      <readonly/></div><div>    </disk></div><div>2. on virsh define if that XML</div><div>2b. You could also "virsh edit" any working xml file, remove the <driver../> which will trigger the same</div><div>2c. You could also define via virt-manager as it does not explicitly specify the device section</div><div><br></div><div>What happens is that before the changes this auto-added a driver section like:</div><div>   <driver name='qemu' type='raw'/></div><div>But now it does only add</div><div>   <driver type='raw'/></div><div>Which fails to verify like:</div><div><br></div><div><br></div><div>Interestingly the same is not true for KVM, there the section is as it was in the past which made it more likely the post-parse step might be involved.</div><div><br></div><div>Also rather expected, if one adds a full <driver name='qemu' type='raw'/> in the XML libvirt doesn't have to provide (broken) defaults and things work as they should - yet from a user with formerly working XMLs or using virt-manager this is a regression.</div><div><br></div><div>I have beg your pardon I seem to not be experienced enough on this part of libvirt to provide a valid fix - I tried but nothing good came out so far. For now I reverted 321a28c6 which also affects a test later added by 4cd3f241. With those changes it will not add anything which lets us start Xen VMs for now, but it clearly isn't a solution.</div><div><br></div><div>I wanted to ask you if you could take a look into this?</div><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><span style="color:rgb(136,136,136);font-size:12.8px">Christian Ehrhardt</span><div style="color:rgb(136,136,136);font-size:12.8px">Software Engineer, Ubuntu Server</div><div style="color:rgb(136,136,136);font-size:12.8px">Canonical Ltd</div></div></div></div></div>
</div>