semantic-conventions icon indicating copy to clipboard operation
semantic-conventions copied to clipboard

[db] Add docs for context info propagation

Open sincejune opened this issue 8 months ago • 2 comments

Relevant to #2162

Changes

Added documentations for instrumentations to propagate context information to databases

Merge requirement checklist

sincejune avatar May 06 '25 14:05 sincejune

@open-telemetry/semconv-db-approvers, PTAL

pellared avatar May 07 '25 10:05 pellared

Added documentations for instrumentations to propagate context information to databases

Should we clarify that this article is specific to MS SQL Server, since context_info is only available there? Other databases like Oracle and Postgres can use fields like v$session.action or application_name to propagate tracecontext.

nenadnoveljic avatar May 12 '25 13:05 nenadnoveljic

Added documentations for instrumentations to propagate context information to databases

Should we clarify that this article is specific to MS SQL Server, since context_info is only available there? Other databases like Oracle and Postgres can use fields like v$session.action or application_name to propagate tracecontext.

@nenadnoveljic I only added the context_info section to the SqlServer's documentation, so I believe we should be fine with that. What do you think?

sincejune avatar May 19 '25 06:05 sincejune

I have refactored the PR to the following structure. @trask @nenadnoveljic @zacharycmontoya PTAL

  • For every database
    • Put service.name on sql comments.
  • For sqlserver (extension)
    • Put traceparent value via SET CONTEXT_INFO (opt-in)

Failed CI is irrelevant to this PR. https://github.com/open-telemetry/semantic-conventions/actions/runs/15172537474/job/42665853199?pr=2236 (fix)

XSAM avatar May 21 '25 21:05 XSAM

This PR was marked stale due to lack of activity. It will be closed in 7 days.

github-actions[bot] avatar Jun 20 '25 03:06 github-actions[bot]

Just saw this and very interested in any effort to clarify the role of sqlcommenter in OTel. @tammy-baylis-swi and i will be sharing our learnings at the Open Observability Summit June 26th, would love to talk about it with any maintainers or TC members there.

cheempz avatar Jun 23 '25 18:06 cheempz

Superseded by #2495

XSAM avatar Jul 07 '25 22:07 XSAM