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

Improve documentation for Opentelemetry inputs #15814

Closed
heliapb opened this issue Sep 2, 2024 · 2 comments
Closed

Improve documentation for Opentelemetry inputs #15814

heliapb opened this issue Sep 2, 2024 · 2 comments
Labels
bug unexpected problem or unintended behavior waiting for response waiting for response from contributor

Comments

@heliapb
Copy link

heliapb commented Sep 2, 2024

Relevant telegraf.conf

[[inputs.opentelemetry]]
    tagexclude = ["service.instance.id" ]

Logs from Telegraf

N/A

System info

Telegraf 1.30.3, CentOS 7

Docker

No response

Steps to reproduce

It would be beneficial to have in the sample.conf for opentelemetry an example how to drop any of the opentelemetry attributes. I was able to do it with the tagexclude as I show in the conf, but was not very straightforward to do. As the documentation seemed to point into using fielddrop, but didn't work for opentelemetry attributes, even those get turned into prometheus labels.

I don't mind adding an example or two for my use case, as I used Java Auto instrumentation with a service, and then used the existing telegraf to collect those metrics.

Expected behavior

N/a

Actual behavior

N/a

Additional info

No response

@heliapb heliapb added the bug unexpected problem or unintended behavior label Sep 2, 2024
@srebhan
Copy link
Member

srebhan commented Sep 2, 2024

@heliapb we do have a general documentation on how to modify data. I don't think it is reasonable to repeat this in the 200+ input plugins. If you do have suggestions on how to improve the general documentation and/or make stuff more discover-able, I'm happy to hear your suggestions!

However, adding general options to the sample.conf file is not what we should do, think about cases where we add (or deprecate) a general option, who should modify all plugins?

@srebhan srebhan added the waiting for response waiting for response from contributor label Sep 2, 2024
@telegraf-tiger
Copy link
Contributor

Hello! I am closing this issue due to inactivity. I hope you were able to resolve your problem, if not please try posting this question in our Community Slack or Community Forums or provide additional details in this issue and reqeust that it be re-opened. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug unexpected problem or unintended behavior waiting for response waiting for response from contributor
Projects
None yet
Development

No branches or pull requests

2 participants