Skip to content

Add knowledge base entries on changelogs and dependency isolation - #9034

Open
jack-berg wants to merge 1 commit into
open-telemetry:mainfrom
jack-berg:knowledge-base-entries
Open

jack-berg wants to merge 1 commit into
open-telemetry:mainfrom
jack-berg:knowledge-base-entries

Conversation

@jack-berg

Copy link
Copy Markdown
Member
  • Changelog advice: don't add since entries are automatically added during the release. Would need to revisit if we add dedicated changelog tooling, but for now this reflects reality.
  • Dependency isolation: reflects how we've implemented functionality which has load bearing dependency on 3rd party library, like OTLP exporters on http / grpc clients, and declarative config on yaml parsing and object conversion. The advice is:
    • Make the dependency pluggable via SPI
    • Name the implementations of the SPI modules to include the dependency major version in the name such that a new module can be published when a new major version of the dependency is published

@jack-berg
jack-berg requested a review from a team as a code owner November 13, 2026 15:36
@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Nov 13, 2026 •

Copy link
Copy Markdown

Pull request dashboard status

Waiting on maintainers · refreshed 2026-11-13 16:11 UTC

Merge when ready.

Status above doesn't look right?
  • Just replied or pushed? Anything around or after the refresh time above may not be picked up yet — give it a few minutes.
  • Anything look wrong? Report it with what you expected; it helps us improve the dashboard.

@codecov

codecov Bot commented Nov 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.45%. Comparing base (4ce8c01) to head (e57655e).

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #9034      +/-   ##
============================================
- Coverage     90.45%   90.45%   -0.01%     
  Complexity    11179    11179              
============================================
  Files          1032     1032              
  Lines         30500    30500              
  Branches       3991     3991              
============================================
- Hits          27590    27589       -1     
  Misses         1935     1935              
- Partials        975      976       +1     

☔ View full report in Codecov by Harness.
📢 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.

This branch has not been deployed

No deployments
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.

2 participants