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

Add OTelLogHandler that sends log records to an OTelLogRecordProcessor #108

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

Joannis
Copy link
Contributor

@Joannis Joannis commented Mar 25, 2024

Copy link

codecov bot commented Mar 25, 2024

Codecov Report

Attention: Patch coverage is 78.04878% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 98.54%. Comparing base (0735233) to head (f9886fe).

Files Patch % Lines
...ogging/Processing/OTelNoOpLogRecordProcessor.swift 0.00% 9 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #108      +/-   ##
==========================================
- Coverage   98.97%   98.54%   -0.44%     
==========================================
  Files          67       70       +3     
  Lines        1951     1992      +41     
==========================================
+ Hits         1931     1963      +32     
- Misses         20       29       +9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@slashmo slashmo changed the title Add OTelLogHandler that sends OTelLogEntry to an OTelLogEntryProcessor Add OTelLogHandler that sends log records to an OTelLogRecordProcessor Mar 27, 2024
@Andrewangeta
Copy link

@slashmo @Joannis Any updates on this one 👀?

@slashmo
Copy link
Owner

slashmo commented Apr 23, 2024

Any updates on this one 👀?
@Andrewangeta

Not yet, unfortunately. I've been busy at work lately, and this needs a bit more work to bring it on par with the OTel spec.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

Add OTelLogRecordProcessor Add swift-log handler
3 participants