[libvirt] [PATCH v2 4/6] tests: utils: Add PreFormat callback for CompareXML2XML helper

Laine Stump laine at laine.org
Tue Feb 9 20:13:32 UTC 2016


On 02/09/2016 10:59 AM, Cole Robinson wrote:
> This allows individual driver tests to hook in their own code before
> the def is formatted and compared.
>
> We will eventually use this in the qemuxml2xml
> ---

Oops. I ACKed 5/6 before this one (after looking ahead to see how this 
would be used).

This looks okay, although I'm wondering why you call just the PCI 
address assignment rather than a full postparse callback. Other things 
can happen in there, e.g. auto-adding controllers. We may as well test 
the whole thing.

Still this is a good start at that, so ACK.





More information about the libvir-list mailing list