Skip to content

Conversation

@trask
Copy link
Member

@trask trask commented Aug 6, 2025

Incubating implementation for open-telemetry/opentelemetry-specification#4612

github-merge-queue bot pushed a commit to open-telemetry/opentelemetry-specification that referenced this pull request Sep 23, 2025
…ers (#4612) Alternative to #4611 Some related prior discussions: - #4208 (comment) - #4207 (comment) Java POC @ open-telemetry/opentelemetry-java#7529 Declarative config for this is a bit nicer than in #4611, also supports applying different minimum severity levels to different loggers: ``` file_format: "1.0" logger_provider: logger_configurator/development: default_config: minimum_severity: WARN trace_based: true loggers: - name: "com.example.app.*" config: minimum_severity: INFO - name: "com.example.db.*" config: minimum_severity: ERROR ``` --------- Co-authored-by: Liudmila Molkova <neskazu@gmail.com> Co-authored-by: Robert Pająk <pellared@hotmail.com>
@trask trask force-pushed the minimum-severity-and-trace-based-log-configuration branch from 747bc91 to f607dc9 Compare October 22, 2025 03:58
@trask trask changed the title POC Add minimum severity and trace-based logger configuration Incubating: Add minimum severity and trace-based logger configuration Oct 22, 2025
@codecov
Copy link

codecov bot commented Oct 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.21%. Comparing base (de48d1b) to head (7aa2b8d).
⚠️ Report is 12 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@ ## main #7529 +/- ## ============================================ + Coverage 90.16% 90.21% +0.04%  - Complexity 7229 7247 +18  ============================================ Files 821 822 +1 Lines 21809 21828 +19 Branches 2136 2139 +3 ============================================ + Hits 19665 19693 +28  + Misses 1472 1469 -3  + Partials 672 666 -6 

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
@trask trask force-pushed the minimum-severity-and-trace-based-log-configuration branch 3 times, most recently from d8f0544 to c471c04 Compare October 22, 2025 17:17
@trask trask force-pushed the minimum-severity-and-trace-based-log-configuration branch from c471c04 to 6d538c3 Compare October 22, 2025 17:36
@trask trask force-pushed the minimum-severity-and-trace-based-log-configuration branch from ff37f15 to 7131c10 Compare October 22, 2025 19:31
@trask trask force-pushed the minimum-severity-and-trace-based-log-configuration branch from 7131c10 to a88392e Compare October 22, 2025 19:34
@trask trask marked this pull request as ready for review October 22, 2025 19:58
@trask trask requested a review from a team as a code owner October 22, 2025 19:58
@trask
Copy link
Member Author

trask commented Nov 13, 2025

@jack-berg now that open-telemetry/opentelemetry-configuration#392 is merged, would you like me to add declarative config support in this PR or in a follow-up?

@jack-berg
Copy link
Member

Followup is great. We need to wait for declarative config to cut a release anyway. No point in holding this up.

@jack-berg jack-berg merged commit 700227a into open-telemetry:main Nov 13, 2025
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants