Skip to content

fix: merge resource attributes into existing tracers#153

Merged
LucasAlvesSoares merged 5 commits into
mainfrom
telemetry-resource-attrs
Jun 10, 2026
Merged

fix: merge resource attributes into existing tracers#153
LucasAlvesSoares merged 5 commits into
mainfrom
telemetry-resource-attrs

Conversation

@LucasAlvesSoares

Copy link
Copy Markdown
Contributor

Description

In auto_instrument, access private "tracer" attributes in order to merge runtime resource attributes into existing trace providers, e.g K8S OTel operator.

Type of Change

Please check the relevant option:

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Code refactoring
  • Dependency update

How to Test

  1. Run auto_instrument() on an agent
  2. Trigger an LLM call so that it emits telemetry
  3. Resource attributes should look exactly the same across all spans

Checklist

Before submitting your PR, please review and check the following:

  • I have read the Contributing Guidelines
  • I have verified that my changes solve the issue
  • I have added/updated automated tests to cover my changes
  • All tests pass locally
  • I have verified that my code follows the Code Guidelines
  • I have updated documentation (if applicable)
  • I have added type hints for all public APIs
  • My code does not contain sensitive information (credentials, tokens, etc.)
  • I have followed Conventional Commits for commit messages

@LucasAlvesSoares LucasAlvesSoares requested a review from a team as a code owner June 10, 2026 02:04
@LucasAlvesSoares LucasAlvesSoares merged commit 608c9ee into main Jun 10, 2026
10 of 11 checks passed
@LucasAlvesSoares LucasAlvesSoares deleted the telemetry-resource-attrs branch June 10, 2026 13:08
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.

2 participants