Skip to content

Conversation

@jeanbisutti
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings December 10, 2025 10:57
@github-actions github-actions bot added the Monitor - Autoconfigure Monitor OpenTelemetry Autoconfigure label Dec 10, 2025
Copy link
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

This PR adds support for exporting database query parameters from OpenTelemetry spans to Azure Monitor telemetry. Query parameters following the db.query.parameter prefix convention are extracted from span attributes and added as properties to the remote dependency telemetry.

  • Adds a constant for the db.query.parameter prefix to align with OpenTelemetry semantic conventions
  • Implements a new helper method to extract and add database query parameters as telemetry properties
  • Integrates the parameter extraction into the database client span processing flow

@jeanbisutti jeanbisutti force-pushed the jean_export_db_query_parameters branch from 5712a1d to 576ec1c Compare December 10, 2025 13:12
@jeanbisutti
Copy link
Contributor Author

jeanbisutti commented Dec 10, 2025

The failing tests should be fixed with #47528

@jeanbisutti jeanbisutti enabled auto-merge (squash) December 12, 2025 15:47
@jeanbisutti jeanbisutti force-pushed the jean_export_db_query_parameters branch from ac37426 to 8796580 Compare December 12, 2025 16:18
@jeanbisutti jeanbisutti merged commit e231454 into main Dec 15, 2025
18 checks passed
@jeanbisutti jeanbisutti deleted the jean_export_db_query_parameters branch December 15, 2025 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Monitor - Autoconfigure Monitor OpenTelemetry Autoconfigure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants