-
Notifications
You must be signed in to change notification settings - Fork 36
Add github.com/openshift/coredns-mdns plugin #14
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
Conversation
|
At the moment the gopkg.toml points to the openshift branch. When we get the results of the CI we'll merge the patch to github.com/openshift/coredns-mdns and update the dep here. |
cybertron
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested locally, works for me.
|
/lgtm |
|
Will this configuration ever be merged upstream? Is there a comparative approach which allows enabling the plugin without carrying a patch to CoreDNS? (This makes me wonder if we should be building and shipping a wrapper entry point to enable use of external plugins — it might even obviate the need for a custom image build of CoreDNS since we would be vendoring...) |
|
Your link discusses the two options I'm aware of for enabling external plugins. Both of them require code changes, either here or in the wrapper. The only way I can see to avoid carrying a patch would be to make this a whole new image and do the patching at image build time, which seems like overkill to me. We have talked about trying to get the mdns plugin upstream, but it's not likely to happen in time for 4.2, which is where we're going to start needing it. |
|
Okay, after much discussion, we've decided to try and catch master up with upstream and then rebase this PR to use Go modules and merge it. Thanks for bearing with us! |
This plugin allows for discovering hosts via mDNS browsing, which means that those nodes are then exposed via normal unicast DNS.
The baremetal platform (IPI for Bare Metal hosts) includes some additional assets on the bootstrap node for automating some infrastructure requirements that would have normally been handled by some cloud infrastructure service. See data/data/bootstrap/baremetal/README.md for more detailed documentation. Co-authored-by: Ben Nemec <bnemec@redhat.com> Co-authored-by: Mark McLoughlin <markmc@redhat.com> Co-authored-by: Russell Bryant <rbryant@redhat.com> Co-authored-by: Stephen Benjamin <stephen@redhat.com> Co-authored-by: Steven Hardy <shardy@redhat.com>
|
/retest |
|
Passed the tests locally: |
|
Thanks for all your work on this! /lgtm |
|
Thanks for letting me collaborate :-) |
Add github.com/openshift/coredns-mdns plugin
Signed-off-by: Antoni Segura Puimedon antoni@redhat.com
1. Why is this pull request needed and what does it do?
This PR adds the coredns-mdns plugin developed to allow coredns to be used in environments without resolution for openshift nodes thanks to its interaction with github.com/openshfit/mdns-publisher.
This should not affect cluster dns operation as the plugin won't be loaded by the cluster-dns-operator Corefile.