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

feat(telemetry): Send logs to OTel collector directly using OTel libraries #2516

Merged

Conversation

calebschoepp
Copy link
Collaborator

image

This is the other way we could do this compared to #2514 .

@calebschoepp calebschoepp force-pushed the logging-signal-basic-libraries branch 2 times, most recently from 8051969 to 3a5f97d Compare May 22, 2024 19:16
@calebschoepp
Copy link
Collaborator Author

I've convinced myself that this is the right approach compared to #2514. @lann / @itowlson this is ready for a review from an implementation perspective.

@calebschoepp calebschoepp marked this pull request as ready for review May 22, 2024 19:17
@calebschoepp calebschoepp force-pushed the logging-signal-basic-libraries branch from 3a5f97d to 993e464 Compare May 27, 2024 16:12
crates/telemetry/src/logs.rs Show resolved Hide resolved
crates/telemetry/src/logs.rs Outdated Show resolved Hide resolved
…aries

Signed-off-by: Caleb Schoepp <caleb.schoepp@fermyon.com>
@calebschoepp calebschoepp force-pushed the logging-signal-basic-libraries branch from 993e464 to 6ff2b5a Compare May 31, 2024 16:17
@calebschoepp calebschoepp merged commit 11e8888 into fermyon:main May 31, 2024
17 checks passed
@calebschoepp calebschoepp deleted the logging-signal-basic-libraries branch May 31, 2024 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants