[libvirt] [PATCH v5 0/5] Add TXT record and hosts support for DNS service and dnsmasq command-line regression testing

[Laine Stump]

On 06/14/2011 09:02 AM, Michal Novotny wrote:
> this is the patch to introduce the TXT record and DNS hosts support
> for the DNS service on the virtual network.

This series is very close, and I'd like to see it go in in time for this 
release (keep in mind that the freeze for new features is sometime 
between Friday and Monday).

There are only two issues left that I haven't provided diffs to fix:

1) Once Patch 2/5 is applied, make check fails. This is solved in Patch 
3/5, but we really need each patch to pass make check, otherwise the 
utility of git bisect is compromised. To fix this, the changes to 
bridge_driver.c in 2/5 and 3/5 need to be combined into a new 2/5, and 
the other changes from those two patches (everything associated with the 
regression tests) needs to be combined into the new 3/5.

2) The data structure for virNetworkDNSHostsDef is not a correct model 
of the data, leading to unnecessarily complicated code. Rather than being:

   struct virNetworkDNSHostsDef {
       virSocketAddr ip;
       char *name;
   } virNetworkDNSHostsDef;

   struct virNetworkDNSHostsDef {
       virSocketAddr ip;
       unsigned int nnames;
       char **names;
   } virNetworkDNSHostsDef;

names will be an array of names, rather than a single name. This will 
eliminate all of the strange loops in virNetworkDNSHostsDefParseXML, and 
make the created hosts file a more accurate image of what is defined in 
the XML.

Once those two items are fixed, we can push this.

Thanks for being so patient!