Skip to content

Export event-name in OTLP Exporter optionally, if the feature-flag is enabled#1994

Merged
lalitb merged 17 commits into
open-telemetry:mainfrom
lalitb:event-name-otlp-feature-flag
Aug 8, 2024
Merged

Export event-name in OTLP Exporter optionally, if the feature-flag is enabled#1994
lalitb merged 17 commits into
open-telemetry:mainfrom
lalitb:event-name-otlp-feature-flag

Conversation

@lalitb

@lalitb lalitb commented Aug 7, 2024

Copy link
Copy Markdown
Member

Fixes #1941

Changes

Added feature flag populate-logs-event-name in opentelemetry-proto and opentelemetry-otlp crates. The feature flag in the opentelemetry-proto crate will be enabled when it is explicitly enabled for the opentelemetry-otlp crate. And this will result in LogRecord::event_name to be populated in the OTLP log attributes.

TBD - Update Changelog.

Merge requirement checklist

  • CONTRIBUTING guidelines followed
  • Unit tests added/updated (if applicable)
  • Appropriate CHANGELOG.md files updated for non-trivial, user-facing changes
  • Changes in public API reviewed (if applicable)

@lalitb
lalitb requested a review from a team August 7, 2024 18:56
Comment thread opentelemetry-proto/Cargo.toml Outdated
Comment thread opentelemetry-proto/Cargo.toml Outdated
Comment thread opentelemetry-proto/src/transform/logs.rs Outdated
Comment thread opentelemetry-otlp/Cargo.toml
@codecov

codecov Bot commented Aug 7, 2024

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 18.75000% with 13 lines in your changes missing coverage. Please review.

Project coverage is 75.1%. Comparing base (e905cf7) to head (582c638).

Files Patch % Lines
opentelemetry-proto/src/transform/logs.rs 27.2% 8 Missing ⚠️
opentelemetry-stdout/src/logs/transform.rs 0.0% 5 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##            main   #1994     +/-   ##
=======================================
- Coverage   75.1%   75.1%   -0.1%     
=======================================
  Files        122     122             
  Lines      20642   20642             
=======================================
- Hits       15505   15503      -2     
- Misses      5137    5139      +2     

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

Comment thread opentelemetry-proto/src/transform/logs.rs
Comment thread opentelemetry-stdout/Cargo.toml

@cijothomas cijothomas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, please add changelog for stdout too.

Comment thread opentelemetry-proto/src/transform/logs.rs
@lalitb

lalitb commented Aug 8, 2024

Copy link
Copy Markdown
Member Author

LGTM, please add changelog for stdout too.

done.

@lalitb
lalitb merged commit 6a16baf into open-telemetry:main Aug 8, 2024
takumi-earth pushed a commit to earthlings-dev/opentelemetry-rust that referenced this pull request Jan 27, 2026
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.

[Log] EventName should be exporter under feature flag only

3 participants