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

Draft: Logs SDK, OTLP exporter #6

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

Conversation

kaylareopelle
Copy link
Owner

@kaylareopelle kaylareopelle commented Oct 9, 2023

This PR contains an implementation of the OpenTelemetry Logs SDK and an OTLP exporter for Ruby. This implementation awaits review and approval from OTel Ruby maintainers/approvers.

The commits in this PR will be cherry-picked to become smaller PRs to keep this work easy to review.
Current PR under consideration: open-telemetry#1611

OpenTelemetry Logs SDK Specification
OpenTelemetry Spec Compliance Matrix - Logs
OpenTelemetry Ruby Logs Project Board

@kaylareopelle kaylareopelle changed the title Draft: Logs SDK Draft: Logs SDK, OTLP exporter Apr 18, 2024
* Pass the current Context to log record processors on emit
* Access the current span within the current Context using
OpenTelemetry::Trace.current_span
* Reorganize attribute lists to match spec order
kaylareopelle and others added 11 commits May 13, 2024 18:46
Now that we instantiate log records in the logger provider, we can
access the log_record_limits when we create a new log record.
Pass the log_record_limits directly from the logger provider to
the log record and apply the limits in the log record class.
…it-updates

Log record processor with emit updates
Invalid attribute classes for keys and values will now emit an error
message

Also, the Internal.valid_key? and Internal.valid_value? methods are used
for validation to align with Span attributes
Warning was emitted about constant being redefined
@kaylareopelle kaylareopelle changed the base branch from main to metrics-everything August 28, 2024 15:29
@kaylareopelle kaylareopelle changed the base branch from metrics-everything to main August 28, 2024 15:29
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.

1 participant