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.

Sign up for GitHub

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

Jump to bottom

DNS forwarder #8

Open
remohammadi opened this issue Jan 7, 2016 · 1 comment · Fixed by #21
Open

DNS forwarder #8

remohammadi opened this issue Jan 7, 2016 · 1 comment · Fixed by #21
Milestone
1.0 - Stable Release

Comments

@remohammadi
Copy link
Contributor

We should also handle dns requests, to be able to resolve node names, which is required by some platforms like kubernetes. Other requests should be forwarded to the given DNS addresses.

If the blacksmith is run in HA mode, the address of the other instances should also be passed to the nodes, and they should respond to dns requests even if they aren't in the leader mode (See #7).
@remohammadi remohammadi changed the title DNS proxy DNS forwarder Jan 16, 2016
@remohammadi remohammadi added this to the 0.8 - Beta Release milestone Jan 19, 2016
@farnasirim farnasirim mentioned this issue Jan 23, 2016
Merged
remohammadi added a commit that referenced this issue Jan 25, 2016
@remohammadi
Pairing with SkyDNS to resolve node names (fixes #8)
8acee39 
Merge remote-tracking branch 'colonelmo/master'

+ Validating IPness of dns addresses flag
@remohammadi
Copy link
Contributor Author

SkyDNS is better to be replaced. We had to fork it for our situation. I believe using CoreDNS as a library inside blacksmith shouldn't be so hard, and it should decrease our total maintenance effort.

So, we should listen on 53/udp, reply for <Hostname>s and also <Hostname>.<ClusterName>.

@remohammadi remohammadi reopened this Aug 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.0 - Stable Release
Development

Successfully merging a pull request may close this issue.

Added skydns. farnasirim/blacksmith
2 participants
@remohammadi @farnasirim