[libvirt] [PATCH 1/6] v6-6: put dnsmasq parameters into a file

[Gene Czarcinski]

On 11/09/2012 07:36 AM, Guido Günther wrote:
> On Thu, Nov 08, 2012 at 04:13:41PM -0500, Gene Czarcinski wrote:
>> >This patch changes how parameters are passed to dnsmasq.  Instead of
>> >being on the command line, the parameters are put into a file (one
>> >parameter per line) and a commandline --conf-file= specifies the
>> >location of the file.  The file is located in the same directory as
>> >the leases file.
> It'd be great if the commit message would state_why_  this is useful.
> Cheers,
Much as it pains me to admit it but you have a valid point.  I have been 
working on this for some time and the usefulness of this is "obvious" to 
me.  While one of the first objectives was to remove the command line 
clutter by moving the dnsmasq parameters into a conf-file, it is with 
some planned (by me) future additions/enhancements where it becomes more 
important.

1. Specify a second conf-file which will initially be allocated as a 
zero-length file.  This file will not be deleted when the network is 
destroyed.  The purpose here is to be able to add log-queries and/or 
log-dhcp parameters to turn on dnsmasq logging.  This can produce a log 
of clutter in syslog so it should be used onlt when necessary. But, when 
it it needed, it can be the only way to figure out what is happening.

2. A previous patch I submitted which was accepted involved adding the 
local=/<domain-name>/ parameter.  With this parameter, dnsmasq will not 
forward queries for the domain it handles. However, currently it will 
forward reverse lookup queries for it subnetwork.  The fix is to add 
more local=/<ip4>.in-addr-arpa/ for IPv6 and local=/<ip6>.ip6.arpa/.  
And "<ip6>.ip6.arpa" expands into a forty four character string.

I also find it a lot easier to look and and understand a conf-file than 
looking at the the command line.

In addition, the rest of the string of submitted updates all assume this 
this one is applied.

And lastly, from what I have observed, configuration files are preferred 
to command line parameters.

I hope this scratches your itch.

Gene