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

DNS volume 2 #271

Merged
merged 11 commits into from
Feb 24, 2017
Merged

DNS volume 2 #271

merged 11 commits into from
Feb 24, 2017

Conversation

mikz
Copy link
Contributor

@mikz mikz commented Feb 24, 2017

  • drop all nginx resolver directives from tests
  • always use search scopes
  • fix parallelelized dns query with no resolvers
  • wrapper with resolver for sockets
  • simplified resolver initialization

Always use DNS search scopes

In scenarios where there are defined search scopes in /etc/resolv.conf we add one empty one to try to resolve naked domains.

Example: search google.internal travis-worker.internal in /etc/resolv.conf

Docker container started with linked redis container. Query to redis would result in no results
because it would try to find redis.google.internal or redis.travis-worker.internal.
We add empty scope so we also try to resolve the naked domain: redis. which works.

@mikz mikz merged commit eb14121 into master Feb 24, 2017
@mikz mikz deleted the dns-vol-2 branch February 24, 2017 10:02
@mikz mikz removed the in progress label Feb 24, 2017
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.

2 participants