Skip to content
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

Adding unbound as dns resolver Pt. 2 [WIP] #640

Closed
wants to merge 5 commits into from

Conversation

muhlemmer
Copy link
Member

This is subsequent work on PR #385, originally submitted by @obi12341. Mainly some cleanup, merge to latest master and DNS testing. This is still a work in progress and this PR should facilitate discussion.

Docker compose

In a regular docker-compose setup, we cannot avoid using static IP. I don't know if this will break anything? Users will not be able to use docker-compose up -d --scale unbound=x. (Don't know if other Mailu components are capable of doing that, never tried).

Docker swarm

It is not (yet) possible in Docker swarm to use a static VIP for a service. (moby/moby#24170) Replicated services rely on the service name to be resolved, which will not work as a DNS entry. This means also that unbound cannot be replicated, unless multiple serve entries are made.

I've tried exposing the DNS server to the host network and set the DNS entry to the host IP, but docker does not allow this kind of connections between containers, it has to by bridge or overlay network.

Testing

The following things are tested:

  1. Resolving of external hostnames
  2. Resolving of Docker hostnames.

No mail testing has been conducted as of yet. First I want to discuss this approach.

obi12341 and others added 5 commits March 8, 2018 15:57
- Don't upgrade the docker-compose file. (Not in the scope of this feature)
- No need to use aliases. Docker already resolves to service names
- Use a fixed IP range, which stays clear of the network ranges used by Docker. (172.xx.0.0/16)
@muhlemmer
Copy link
Member Author

Closing. We will integrate this in setup as an optional.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants