Skip to content

Flatten stdouttrace Exporter.initSelfObservability into Exporter.New#7197

Merged
MrAlias merged 3 commits intoopen-telemetry:mainfrom
MrAlias:flatten-init-stdouttrace
Aug 17, 2025
Merged

Flatten stdouttrace Exporter.initSelfObservability into Exporter.New#7197
MrAlias merged 3 commits intoopen-telemetry:mainfrom
MrAlias:flatten-init-stdouttrace

Conversation

@MrAlias
Copy link
Copy Markdown
Contributor

@MrAlias MrAlias commented Aug 14, 2025

  • Do not use side-effect programming to setup an exporter. Make the setup explicit, unique, and local.
  • Report instrument creation errors back to the user instead of passing them to otel.Handle.

@MrAlias MrAlias added Skip Changelog PRs that do not require a CHANGELOG.md entry pkg:exporter:stdout Related to the stdout exporter package labels Aug 14, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Aug 14, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.9%. Comparing base (2ab7d4d) to head (c43244b).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #7197   +/-   ##
=====================================
  Coverage   82.9%   82.9%           
=====================================
  Files        264     264           
  Lines      24628   24630    +2     
=====================================
+ Hits       20424   20433    +9     
+ Misses      3821    3817    -4     
+ Partials     383     380    -3     
Files with missing lines Coverage Δ
exporters/stdout/stdouttrace/trace.go 83.6% <100.0%> (+9.1%) ⬆️

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@MrAlias MrAlias marked this pull request as ready for review August 14, 2025 21:13
@MrAlias MrAlias merged commit ee6d57b into open-telemetry:main Aug 17, 2025
37 checks passed
@MrAlias MrAlias deleted the flatten-init-stdouttrace branch August 17, 2025 15:37
@MrAlias MrAlias added this to the v1.38.0 milestone Aug 20, 2025
@MrAlias MrAlias mentioned this pull request Aug 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg:exporter:stdout Related to the stdout exporter package Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants