Skip to content

Add override for tomcat-jdbc scope name#18860

Open
jaydeluca wants to merge 1 commit into
open-telemetry:mainfrom
jaydeluca:tomcat-name
Open

Add override for tomcat-jdbc scope name#18860
jaydeluca wants to merge 1 commit into
open-telemetry:mainfrom
jaydeluca:tomcat-name

Conversation

@jaydeluca
Copy link
Copy Markdown
Member

Related to #18857 (comment)

Copilot AI review requested due to automatic review settings May 27, 2026 13:29
@jaydeluca jaydeluca requested a review from a team as a code owner May 27, 2026 13:29
entry("io.opentelemetry.runtime-telemetry", singleton("test")));
entry("io.opentelemetry.runtime-telemetry", singleton("test")),
// tomcat-jdbc was renamed to tomcat-jdbc-8.5 but the scope name was not changed
entry("io.opentelemetry.tomcat-jdbc-8.5", singleton("io.opentelemetry.tomcat-jdbc")));
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

alternatively we could update the actual scope name, but I wasn't sure if we've been doing that as part of #18428

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@trask should we put the update to the scope name behind v3 preview flag or can we do it right away?

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the instrumentation-docs telemetry parsing allow-list to account for the tomcat-jdbctomcat-jdbc-8.5 module rename where the emitted telemetry scope name remained io.opentelemetry.tomcat-jdbc. This prevents the docs telemetry validation from incorrectly excluding signals coming from the renamed module.

Changes:

  • Add a scope allow-list override mapping io.opentelemetry.tomcat-jdbc-8.5 to permit telemetry emitted under io.opentelemetry.tomcat-jdbc.

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.

3 participants