-
Notifications
You must be signed in to change notification settings - Fork 4.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
can't disable DNS server in consul v0.8.4 with DNS port < 0 #3135
Comments
Thanks @TheLorax this looks like a possible regression - will look at this for the next release. |
Try setting the port to |
We'll fix it to require the port to be |
if I understand this fix correctly, it is no longer possible to not bind the DNS listen port in consul v0.8.4? |
Prior to consul 0.8.4 you could turn off DNS by setting the port to a
number <=0. There was a regression in the 0.8.4 release that made
a DNS disable conditional be port == 0 instead of port <=0, but that conflated with the way a 0 value was interpreted when merging configs, port 0 got turned into 8600 so effectively in the consul 0.8.4 binary there is no way to turn off DNS.
The fix for issue #3135 that's in master but not in the released 0.8.4 binary gets it back to how it worked before - setting port to -1 should cause DNS to be disabled.
|
When I tried setting the port to 0 in v0.8.4 the DNS port was bound and the service was enabled, but on the default DNS port number. (8400 I think) |
@TheLorax you are right, I was able to reproduce this in master setting dns to 0 using the above config (it still started it on port 8600). Setting it to -1 worked as expected and it didn't start a DNS server. For now, could you set your port to -1, (building consul from source). |
I updated my comment above. I am relatively new, so apologies for the back and forth on this. If this is blocking you, I would recommend building consul master branch from source to be able to use your existing config file with |
@preetapan is correct. I assumed |
consul version
for both Client and ServerClient: none
Server: v0.8.4
Server:
The text was updated successfully, but these errors were encountered: