]> git.ipfire.org Git - thirdparty/libvirt.git/commitdiff
docs: schema: make disk driver name attribute optional
authorJim Fehlig <jfehlig@suse.com>
Tue, 18 Jul 2017 16:20:35 +0000 (10:20 -0600)
committerJim Fehlig <jfehlig@suse.com>
Tue, 18 Jul 2017 17:39:52 +0000 (11:39 -0600)
/domain/devices/disk/driver/@name is not a required or mandatory
attribute according to formatdomain, and indeed it was agreed on
IRC that the attribute is "optional for input, recommended (but
not required) for output". Currently the schema requires the
attribute, causing virt-xml-validate to fail on disk config where
the driver name is not explicitly specified. E.g.

# cat test.xml | grep -A 5 cdrom
    <disk type='file' device='cdrom'>
      <driver type='raw'/>
      <target dev='hdb' bus='ide'/>
      <readonly/>
      <address type='drive' controller='0' bus='0' target='0' unit='1'/>
    </disk>

# virt-xml-validate test.xml
Relax-NG validity error : Extra element devices in interleave
test.xml:21: element devices: Relax-NG validity error : Element domain failed to validate content
test.xml fails to validate

Relaxing the name attribute to be optional fixes the validation

# virt-xml-validate test.xml
test.xml validates

docs/schemas/domaincommon.rng

index fc1a40f96f1a72d7a5aad8159cc917eb208f79e2..a49ce9303eb896f9e8fa8b5f025dacf871db6a1c 100644 (file)
     </element>
   </define>
   <define name="driverFormat">
-    <attribute name="name">
-      <ref name="genericName"/>
-    </attribute>
+    <optional>
+      <attribute name="name">
+        <ref name="genericName"/>
+      </attribute>
+    </optional>
     <optional>
       <attribute name='type'>
         <choice>