Skip to content
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.

FDP can't coexist with new DNS server #1215

Closed
dpw opened this issue Jul 27, 2015 · 5 comments
Closed

FDP can't coexist with new DNS server #1215

dpw opened this issue Jul 27, 2015 · 5 comments
Assignees
Milestone

Comments

@dpw
Copy link
Contributor

dpw commented Jul 27, 2015

The new DNS server is integrated into the router, and binds to port 53. FDP runs the router with --net=host, because it needs to perform ODP operations in the default netns. Put them together, and the router container now tries to bind to port 53 in the default netns, and EADDRINUSE ensues. So FDP is now likely to only work if you do weave launch --no-dns.

Either the DNS server needs to be run on some other port number, or it should be disentangled from the router and moved into its own container once again. The latter seems preferable, in order to minimize the surface area of code run in a container with --privileged --net=host.

@rade
Copy link
Member

rade commented Jul 27, 2015

run on some other port number

It actually just needs to listen on a specific interface/ip, namely the docker bridge ip. That's what the port mappings in the script do, but they are ineffective when running with --net=host.

@tomwilkie
Copy link
Contributor

We can add a new argument (--dns-listen-address or something) and have the weave script put the docker bridge ip in there. Will that be enough?

@tomwilkie tomwilkie self-assigned this Jul 28, 2015
@rade rade added this to the current milestone Jul 28, 2015
@tomwilkie tomwilkie assigned dpw and unassigned tomwilkie Jul 29, 2015
@rade
Copy link
Member

rade commented Jul 29, 2015

Why is this still open? I thought #1225 was meant to address this issue?

@tomwilkie
Copy link
Contributor

#1225 mostly addresses the issue; @dpw will need to update the weave script in the FDP branch to launch with the new parameter. I can do this if you like?

@rade
Copy link
Member

rade commented Jul 30, 2015

I am quite sure @dpw considers this issue to be pertaining to the current weave code base only, i.e. making any changes to FDP is out of scope.

@rade rade closed this as completed Aug 4, 2015
@rade rade modified the milestones: current, 1.1.0 Aug 5, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants