-
Notifications
You must be signed in to change notification settings - Fork 28
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
Add support for egress logs in the Otelcollector client #589
Comments
🙌 We're very happy for support for logs to be added, if you'd like to contribute via PR! |
I forgot to mention before, but my preferred solution for getting logs into the OTel Collector would actually be to build a "loggregatorreceiver". See cloudfoundry/otel-collector-release#25. |
Oh wow!! Tomorrow I will have a look at your implementation, but it sounds similar to what we are trying to achieve. We already have submitted a PR (accepted) to the OtelCollector codebase cloudfoundryreceiver to read the logs. We are going to do another PR to generate traces from RTR logs. Because of the amount of logs we have, we would prefer trying an approach similar to what we have in K8S with
Apart of that there would be another set of OTEL collectors which would deal with:
For the points 2 and 3 we would need another OTEL processor similar to K8Sattributes processor which would enrich telemetry taking the appid and querying the CF API for extra metadata (we will implement something basic in a second PR for the Right now our highest priority is dealing with app metrics and logs in a way which would be really simple for end users (adding annotations to apps) and check if this whole idea performs and works as expected, after that we would think about the "CFAttributes processor" which needs to be properly implemented because of the amount of CF API calls it can generate. |
Right now ,there is only support for forwarding envelope metrics to an Otel collector.:
loggregator-agent-release/src/pkg/otelcolclient/otelcolclient.go
Lines 116 to 123 in 07c2467
What do you think about adding support for logs?
We are happy to contribute if a PR is welcome.
The text was updated successfully, but these errors were encountered: