[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH] (updated) additional parameters needed for dnsmasq

On 08/22/2012 01:47 PM, Eric Blake wrote:
On 08/22/2012 11:39 AM, Eric Blake wrote:
On 08/22/2012 10:59 AM, Gene Czarcinski wrote:
As I said in a previous message, dnsmasq is forwarding a number of
queries upstream that should not be done.  There still remains an MX
query for a plain name with no domain specified that will be forwarded
is dnsmasq has --domain=xxx  --local=/xxx/ specified. This does not
happen with no domain name and --local=// ... not a libvirt problem.

ACK and pushed with the above tweak, and with adding you to AUTHORS (let
us know if you prefer any other spelling or email address; the file is
in UTF-8).
Oh, and now that I've already pushed, I have a high-level question: what
is the minimum version of 'dnsmasq' that supports the command-line
syntax that this patch introduces?

+--local=// --domain-needed --filterwin2k \
I honestly do not know for sure ... I am running dnsmasq-2.59 and checking the dnsmasq website shows that the latest is 2.63. The oldest "active" tarball is 2.45 and the example dnsmasq.conf it it has local= domain-needed and filterwin2k

I believe these have been around for some time and no version that libvirt would support will not support them.

BTW, I believe that you folks are making the right choice in using the long-names form of the options ... the single character versions can just be too confusing: "-s" and "-S" mean two very different things. "-s" is the same as "--domain" and "-S" is the same as "--server" which is also the same as "--local".

I am not absolutely sure that "--filterwin2k" is needed to get rid of the bad/undesired forwarding but it sounded like an option that should be used in any case.

If older dnsmasq doesn't recognize --local=// or the new --domain-needed
or --filterwin2k options, then we either need to make this code
conditional based on probing 'dnsmasq --help' at startup, or else change
the spec file to require a larger minimum version of dnsmasq (we already
require 2.41 for other reasons).

I went back and checked some of the older versions of dnsmasq (2.41 and 2.37). local=//, --domain-needed, and --filterwin2k are all present in the example dnsmasq.conf files. This is not to say the the dnsmasq software does exactly the same thing in all versions. According the the CHANGELOG, there was some code "tweaking" for domain-needed in 2.58.

I am truly sorry that my patch and email made life a bit difficult for you. I have not delved into any of the virtualization code and your setup & convensions are all reasonable but I just did not know. The last package I hacked around with was NetworkManager and libvirt is a lot different.

As far as the code change you made to my patch, both versions work but yours is a bit more eligant.


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]