Skip to content

add config bridge based on ExtendedOpenTelemetry #14767

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 11 commits into
base: main
Choose a base branch
from
Draft

add config bridge based on ExtendedOpenTelemetry #14767

wants to merge 11 commits into from

Conversation

@zeitlinger
Copy link
Member

@zeitlinger zeitlinger commented Sep 23, 2025 •
edited
Loading

Fixes #14192

@zeitlinger zeitlinger self-assigned this Sep 23, 2025
@github-actions github-actions bot added the test native This label can be applied to PRs to trigger them to run native tests label Sep 23, 2025
@zeitlinger zeitlinger mentioned this pull request Sep 23, 2025
Open
@otelbot-java-instrumentation
Copy link
Contributor

otelbot-java-instrumentation bot commented Sep 23, 2025

🔧 The result from generateLicenseReport was committed to the PR branch.

@zeitlinger zeitlinger mentioned this pull request Sep 25, 2025
Closed
@zeitlinger zeitlinger mentioned this pull request Oct 7, 2025
Open
@zeitlinger zeitlinger force-pushed the simple-config-bridge branch from 76e3cf1 to 38fa604 Compare October 29, 2025 09:14
@zeitlinger
Copy link
Member Author

zeitlinger commented Oct 31, 2025

from SIG meeting:

try using declarative config API directly instead of a bridge, because there are not so many usages that are affected

@zeitlinger zeitlinger mentioned this pull request Oct 31, 2025
Open
@zeitlinger
Copy link
Member Author

zeitlinger commented Oct 31, 2025

from SIG meeting:

try using declarative config API directly instead of a bridge, because there are not so many usages that are affected

created #15199 to try it out

@zeitlinger zeitlinger force-pushed the simple-config-bridge branch from 7edea17 to 9e59627 Compare November 9, 2025 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@zeitlinger zeitlinger

Labels

test native This label can be applied to PRs to trigger them to run native tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

use InstrumentationConfig where possible

1 participant

@zeitlinger