Skip to content

[receiver/sqlserver] Use shared resource builder for logs and metrics#46740

Merged
songy23 merged 6 commits into
open-telemetry:mainfrom
XSAM:unify-resource-builder-for-sqlserver
Mar 10, 2026
Merged

[receiver/sqlserver] Use shared resource builder for logs and metrics#46740
songy23 merged 6 commits into
open-telemetry:mainfrom
XSAM:unify-resource-builder-for-sqlserver

Conversation

@XSAM

@XSAM XSAM commented Mar 7, 2026

Copy link
Copy Markdown
Member

Description

This unify the resource attribute logic for both logs and metrics. It also fixes the #42355 where the PR #46484 left logs part behind.

Link to tracking issue

Fixes #42355

@XSAM XSAM marked this pull request as ready for review March 7, 2026 00:46
@XSAM XSAM requested review from a team and crobert-1 as code owners March 7, 2026 00:46
@github-actions github-actions Bot added receiver/sqlserver Run Windows Enable running windows test on a PR labels Mar 7, 2026
@github-actions github-actions Bot requested a review from sincejune March 7, 2026 00:46
@crobert-1 crobert-1 added the ready to merge Code review completed; ready to merge by maintainers label Mar 10, 2026
@songy23 songy23 merged commit eddfe34 into open-telemetry:main Mar 10, 2026
277 of 279 checks passed
@XSAM XSAM deleted the unify-resource-builder-for-sqlserver branch March 30, 2026 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready to merge Code review completed; ready to merge by maintainers receiver/sqlserver Run Windows Enable running windows test on a PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[receiver/sqlserver] host.name resource attribute is not set properly when using datasource configuration

5 participants