[libvirt] [PATCH] nodedev: Resolve Relax-NG validity error

Commit id 'c4a4603de' added an output <path> to the nodedev xml, but did not update the schema. This resulted in the failure of the 'virt-xml-validate' on a file generated by 'virsh nodedev-dumpxml pci_0000_00_00_0' (for example). This was found/seen by running autotest on my host. --- I'm not sure if something like this needs to be backported or not, but I'm sure someone will let me know... docs/schemas/nodedev.rng | 3 +++ 1 file changed, 3 insertions(+) diff --git a/docs/schemas/nodedev.rng b/docs/schemas/nodedev.rng index d2bebff..81ab4d4 100644 --- a/docs/schemas/nodedev.rng +++ b/docs/schemas/nodedev.rng @@ -13,6 +13,9 @@ and in virsh --> <element name="name"><text/></element> <optional> + <element name="path"><text/></element> + </optional> + <optional> <element name="parent"><text/></element> </optional> -- 1.8.3.1

On 10/03/2013 03:16 PM, John Ferlan wrote:
Commit id 'c4a4603de' added an output <path> to the nodedev xml, but did not update the schema.
This resulted in the failure of the 'virt-xml-validate' on a file generated by 'virsh nodedev-dumpxml pci_0000_00_00_0' (for example).
This was found/seen by running autotest on my host. --- I'm not sure if something like this needs to be backported or not, but I'm sure someone will let me know...
docs/schemas/nodedev.rng | 3 +++ 1 file changed, 3 insertions(+)
ACK. At the moment, I wouldn't worry about backporting it unless someone opens a bugzilla complaining about hitting the failure. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org

On 10/03/2013 07:18 PM, Eric Blake wrote:
On 10/03/2013 03:16 PM, John Ferlan wrote:
Commit id 'c4a4603de' added an output <path> to the nodedev xml, but did not update the schema.
This resulted in the failure of the 'virt-xml-validate' on a file generated by 'virsh nodedev-dumpxml pci_0000_00_00_0' (for example).
This was found/seen by running autotest on my host. --- I'm not sure if something like this needs to be backported or not, but I'm sure someone will let me know...
docs/schemas/nodedev.rng | 3 +++ 1 file changed, 3 insertions(+)
ACK. At the moment, I wouldn't worry about backporting it unless someone opens a bugzilla complaining about hitting the failure.
Pushed John
participants (2)
-
Eric Blake
-
John Ferlan