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_query should query all servers in parallel #2547

Closed
phemmer opened this issue Mar 18, 2017 · 1 comment · Fixed by #4754
Closed

dns_query should query all servers in parallel #2547

phemmer opened this issue Mar 18, 2017 · 1 comment · Fixed by #4754
Labels
feature request Requests for new plugin and for new features to existing plugins
Milestone

Comments

@phemmer
Copy link
Contributor

phemmer commented Mar 18, 2017

Feature Request

Proposal:

The dns_query input should query all servers in parallel.

Current behavior:

The dns_query plugin queries all server sequentially.

Desired behavior:

The dns_query input should query all servers in parallel.

Use case: [Why is this important (helps with prioritizing requests)]

When diagnosing issues, having all the servers queried at the same time is vitally important. For example if you're querying 2 different DNS servers with a recursive lookup, and there is an issue with the recursion upstream, both servers will be affected. However if it is only a momentary issue, telegraf is going to hit the first DNS server, it'll stall out during recursion until the momentary issue passes, and then it'll proceed to the second server which will return fast because the upstream issue has passed. Thus the first DNS server will appear to have a problem, when really the problem is just visible on whatever server is first in the list.

@danielnelson
Copy link
Contributor

👍

@Jaeyo Jaeyo mentioned this issue Sep 26, 2018
3 tasks
@danielnelson danielnelson added this to the 1.9.0 milestone Sep 28, 2018
@danielnelson danielnelson added the feature request Requests for new plugin and for new features to existing plugins label Sep 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request Requests for new plugin and for new features to existing plugins
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants