[libvirt] [PATCH 2/2] Network: Add support for DNS hosts definition

[Michal Novotny]

On 03/31/2011 01:14 PM, Paolo Bonzini wrote:
> On 03/31/2011 01:06 PM, Michal Novotny wrote:
>>>>  - this would be the third patch in the series, and it would add
>>>>  dnsmasqAddHost calls in networkSaveDnsmasqHostsfile
>> So this patch will be just adding the networkSaveDnsmasqHostsfile() call
>> to the dnsmasqAddHost() and nothing else ?
> And all the XML processing.


Oh, ok. That's fine :) I'll create put it there as well.



>>>>>>  diff --git a/tests/networkxml2xmlin/nat-network-dns-hosts.xml b/tests/networkxml2xmlin/nat-network-dns-hosts.xml
>>>>>>  new file mode 100644
>>>>>>  index 0000000..fe545cf
>>>>  Uhm, libvirt has no tests that actually check whether the dnsmasq
>>>>  command line works?  That's a bit bad, perhaps you can add them...
>> Well, unfortunately there are no tests to check whether the dnsmasq
>> command line works. What would you prefer? To run it with some bogus PID
>> file and port and check the command execution error code and if it's 0
>> (and PID is working) then to kill $PID and make the test pass, otherwise
>> fail the test or something similar?
> Even a regression test that compares the command-line with what is in a 
> file, would be great.

You mean to compare command-line from the XML, i.e. to generate the
command-line from XML and put the tests whether the generated
command-line is the same as mentioned in the *.argv file containing
whole command-line? Like for QEMU?

Michal


-- 
Michal Novotny <minovotn at redhat.com>, RHCE
Virtualization Team (xen userspace), Red Hat