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

Problems to use otel-collector as log pipeline #4955

Closed
gillg opened this issue Aug 29, 2021 · 5 comments
Closed

Problems to use otel-collector as log pipeline #4955

gillg opened this issue Aug 29, 2021 · 5 comments

Comments

@gillg
Copy link
Contributor

gillg commented Aug 29, 2021

Use case sample
Syslogreceiver > Lokiexporter

Is your feature request related to a problem? Please describe.
Hello, as discussed in previous issues (I will search and attach them later) otel logs can have a structured body, and/or attributes. In different case we have very different results.
Ex: fluent forward receiver don't fill any body... We only have attributes.
In opposite, syslogreceiver don't fill attributes and we only have (a structured) body.

Both receivers are correct in term of opentelemtry.
But at the exporter level it can be complex to handle (here impossible for syslog) depending on the technology.
Loki need a body and labels (selected set on metadata), so we need a way to extract them from otel attributes, and/or structured body, and/or resources.

An additional processor to parse an unstructured body could be useful in some cases (even if I'm not fan for perf reasons), to convert it as structured or map extracted parts to attributes.

@punya punya added the data:logs Logs related issues label Sep 9, 2021
@punya
Copy link
Member

punya commented Sep 9, 2021

Hi @gillg! Would you mind linking the previous issues relevant to this topic? Also, it would be helpful to call out specific enhancements that you're looking for, with as much precision and concrete use cases as possible.

@gillg
Copy link
Contributor Author

gillg commented Sep 9, 2021

Hi @punya thanks for the reminder ! 😅
A similar issue / brainstorming was on #14718
It's initially related to #2529

A PR is in progress on Loki exporter to allow a better mapping between otel / Loki but it was not plan to be able to map body attributes to Loki labels (specific log metadata similar to attributes but to chose carefully, at least one is mandatory)

I also started some rewrite (#5173) on fluent receiver to allow a manual mapping from attributes to body and severity.

@gillg
Copy link
Contributor Author

gillg commented Sep 9, 2021

Maybe, an alternative to these pr could be to extend the attributesprocessor (or a new thing) to manipulate a structured body as AttributeMap and be able to move data from attributes to body and vice versa.

hex1848 pushed a commit to hex1848/opentelemetry-collector-contrib that referenced this issue Jun 2, 2022
This new template can be used when creating a new release.

Fixes open-telemetry#4883
@github-actions
Copy link
Contributor

github-actions bot commented Dec 5, 2022

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

@github-actions github-actions bot added the Stale label Dec 5, 2022
@github-actions
Copy link
Contributor

This issue has been closed as inactive because it has been stale for 120 days with no activity.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Mar 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants