feat(otlp): Add docs for OTLP logs endpoint #14898
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
Bundle ReportChanges will increase total bundle size by 125 bytes (0.0%) ⬆️. This is within the configured threshold ✅ Detailed changes
Affected Assets, Files, and Routes:view changes for bundle: sentry-docs-server-cjsAssets Changed:
Files in
view changes for bundle: sentry-docs-client-array-pushAssets Changed:
Files in
|
AbhiPrasad
force-pushed
the
abhi-otel-logs-docs
branch
from
7353b7f to
844a601
Compare
![seer-by-sentry[bot]](https://avatars.githubusercontent.com/in/801464?s=60&v=4){kind=small}
docs/concepts/otlp/index.mdx
Outdated
| processors: [new BatchRecordProcessor(logExporter)], | ||
| }); |
There was a problem hiding this comment.
Potential bug: The OTLP logs documentation example uses BatchRecordProcessor, but the correct imported class is BatchLogRecordProcessor, which will cause a runtime error.
-
Description: The code example for setting up OpenTelemetry logs imports the class
BatchLogRecordProcessorbut then attempts to instantiateBatchRecordProcessor. This mismatch will cause aReferenceError: BatchRecordProcessor is not definedwhen a user copies and runs the provided code. This prevents the code example from working and will cause an immediate crash when trying to initialize theLoggerProvider. -
Suggested fix: In the
LoggerProviderinstantiation, change the incorrect class nameBatchRecordProcessorto the correct, imported class nameBatchLogRecordProcessor.
severity: 0.7, confidence: 0.98
Did we get this right? 👍 / 👎 to inform future reviews.
|
|
||
| If you have an existing OpenTelemetry log instrumentation, you can configure your OpenTelemetry exporter to send logs to Sentry directly. Sentry's OTLP ingestion logs endpoint has the following known limitations: | ||
|
|
||
| - Array attributes are partially supported. We ingest and display array attributes, but they cannot be searched, filtered, or aggregated. |
There was a problem hiding this comment.
Do we expect this list of limitations to grow? If not, I'd recommend not displaying this in a list format.
There was a problem hiding this comment.
yes the list will grow, so I'll keep it like this for now.
AbhiPrasad
force-pushed
the
abhi-otel-logs-docs
branch
from
76d8a16 to
38ff597
Compare
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
resolves https://linear.app/getsentry/issue/LOGS-347/add-docs-for-otlp-logs-endpoint