-
Notifications
You must be signed in to change notification settings - Fork 62
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 log record exporter #589
Comments
@aabmass I couldn't find anything else that does this and therefore created this feature request. Please let me know if I am missing something. |
@pksunkara thanks for the request. Hoping you can provide some more context, is there a reason that sending OTLP logs to the Collector, Logging to stdout/stderr or a file wouldn't work for you? |
Sending OTLP logs to the:
We have some small services that are hosted on PaaS apps outside GCP, that we want to send the logs to our central monitoring solution in GCP. |
@aabmass Any update on this? GCP wouldn't work with opentelemetry without this. |
Hey apologies I missed your last message. I'll talk it over with the team and get back to you. It's definitely low priority on our side since there are existing workarounds. |
@aabmass I would be willing to contribute this if you can point me on how to get started and tell me a bit on how some internals work. |
@pksunkara unfortunately we can't accept a PR for this to this repo as it's not something we want to maintain. Of course you are welcome to implement it on your own and handle maintenance. Here is the code for our collector Logs exporter https://github.com/GoogleCloudPlatform/opentelemetry-operations-go/blob/main/exporter/collector/logs.go |
Is your feature request related to a problem? Please describe.
There is currently no way to export log records produced by OpenTelemetry to GCP Cloud Logging in pure JS land.
Describe the solution you'd like
Add a
LogRecordExporter
that extends OpenTelemetry's JSLogRecordExporter
that exports the logs to GCP Cloud LoggingThe text was updated successfully, but these errors were encountered: