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

datadog output plugin changes "_" -> "." #1024

Closed
jwerak opened this issue Apr 14, 2016 · 2 comments · Fixed by #1063
Closed

datadog output plugin changes "_" -> "." #1024

jwerak opened this issue Apr 14, 2016 · 2 comments · Fixed by #1063
Labels
bug unexpected problem or unintended behavior

Comments

@jwerak
Copy link

jwerak commented Apr 14, 2016

Datadog output plugin changes "_" -> "." (https://github.com/influxdata/telegraf/blob/master/plugins/outputs/datadog/datadog.go#L74) and there is no option to change this behavior.

It would be nice to have switch to disable this, also default should be that nothing is changed.

Also I am not following influx world for long, but it would IMO make sense to edit metrics on input only.
@jwerak jwerak mentioned this issue Apr 14, 2016
Closed
@sparrc sparrc added the bug unexpected problem or unintended behavior label Apr 15, 2016
sparrc added a commit that referenced this issue Apr 20, 2016
@sparrc
Don't replace _ with . in datadog names
9478776 
closes #1024
@sparrc sparrc mentioned this issue Apr 20, 2016
Merged
sparrc added a commit that referenced this issue Apr 20, 2016
@sparrc
Don't replace _ with . in datadog names
4bcf157 
closes #1024
@hauleth
Copy link

hauleth commented Jul 26, 2017

The documentation wasn't updated since then and I think it should be an option.

@danielnelson
Copy link
Contributor

@hauleth Can you open a new issue with the feature request?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug unexpected problem or unintended behavior
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Don't replace _ with . in datadog names influxdata/telegraf
4 participants
@hauleth @jwerak @danielnelson @sparrc