<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jul 15, 2017 at 12:27 AM, Jim Fehlig <span dir="ltr"><<a href="mailto:jfehlig@suse.com" target="_blank">jfehlig@suse.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 07/11/2017 08:15 AM, Christian Ehrhardt wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br></blockquote><span class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
What happens is that before the changes this auto-added a driver section like:<br>
    <driver name='qemu' type='raw'/><br>
But now it does only add<br>
    <driver type='raw'/><br>
Which fails to verify like:<br>
</blockquote>
<br></span>
I think you forgot to paste the example here, but I assume it is something like<br>
<br>
# virt-xml-validate test.xml<br>
Relax-NG validity error : Extra element devices in interleave<br>
test.xml:21: element devices: Relax-NG validity error : Element domain failed to validate content<br>
test.xml fails to validate<br></blockquote><div> </div><div>Yes that was the case I've seen.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
where test.xml contains disk config<span class=""><br>
<br>
    <disk type='file' device='cdrom'><br></span>
      <driver type='raw'/><span class=""><br>
      <target dev='hdb' bus='ide'/><br>
      <readonly/><br></span>
      <address type='drive' controller='0' bus='0' target='0' unit='1'/><br>
    </disk><br></blockquote><div><br></div><div>As your example or even no <driver> at all - that in the past also got the defaults and no isn't.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
It is not clear to me if 'name' is a required attribute of <disk>. The doc says "If the hypervisor supports multiple backend drivers, then the name attribute selects the primary backend driver name". It doesn't mention "required" or "mandatory". We'd have a schema bug if 'name' is optional. I guess it is a good question for danpb. Dan, is disk/driver/@name a required attribute?<br></blockquote><div><br></div><div> Prepping my patch, but surely if the fix is to just relax the relaxng to not have it required I'm good as well.</div></div>
</div></div>