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

Fix ntpq parse issue when using dns_lookup #3026

Merged
merged 3 commits into from
Jul 18, 2017
Merged

Fix ntpq parse issue when using dns_lookup #3026

merged 3 commits into from
Jul 18, 2017

Conversation

dsalbert
Copy link
Contributor

Add regular expression to tackle a problem #2386
Change:

  • Add regular expression to remove unwanted string
  • Create Unit Test for this case

Required for all PRs:

  • Signed CLA.
  • Associated README.md updated.
  • Has appropriate unit tests.

@danielnelson danielnelson merged commit d2626f1 into influxdata:master Jul 18, 2017
@danielnelson
Copy link
Contributor

Thanks!

jeichorn pushed a commit to jeichorn/telegraf that referenced this pull request Jul 24, 2017
@jeichorn
Fix ntpq parse issue when using dns_lookup (influxdata#3026)
21e367f
@danielnelson danielnelson added this to the 1.3.5 milestone Jul 25, 2017
@danielnelson danielnelson changed the title Fix issue #2386 Fix ntpq parse issue when using dns_lookup Jul 25, 2017
danielnelson pushed a commit that referenced this pull request Jul 25, 2017
@danielnelson
Fix ntpq parse issue when using dns_lookup (#3026)
d4f59a1 
(cherry picked from commit d2626f1)
maxunt pushed a commit that referenced this pull request Jun 26, 2018
@danielnelson
Fix ntpq parse issue when using dns_lookup (#3026)
9a8de60
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.3.5
Development

Successfully merging this pull request may close these issues.
2 participants
@dsalbert @danielnelson