Skip to content

fix: remove OpenInference external span provider registration#750

Closed
cfauchere wants to merge 1 commit intomainfrom
fix/remove-openinference-external-span-provider
Closed

fix: remove OpenInference external span provider registration#750
cfauchere wants to merge 1 commit intomainfrom
fix/remove-openinference-external-span-provider

Conversation

@cfauchere
Copy link
Copy Markdown
Contributor

Summary

  • Stop registering OpenInference as the external span provider in the langchain runtime factory
  • The agent runtime (uipath-agents-python) registers its own provider via SpanHierarchyManager, which returns the correct LLMOps span
  • Having OpenInference also registered caused trace ID mismatches on alpha where the OTEL context didn't propagate through LangGraph's async tasks
  • Companion PR: fix(platform): use LLMOps span ID in trace header uipath-python#1547

Test plan

  • Local validation: ran agent with CG tool, verified header span ID matches the LLMOps trace span
  • Trace ID correct across all headers
  • No Failed to detach context errors
  • Agent completes successfully

🤖 Generated with Claude Code

Stop registering OpenInference as the external span provider in the
langchain runtime factory. The agent runtime (uipath-agents-python)
registers its own provider via SpanHierarchyManager, which returns the
correct LLMOps span. Having OpenInference also registered caused trace
ID mismatches on alpha where the OTEL context didn't propagate through
LangGraph's async tasks.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@cfauchere cfauchere force-pushed the fix/remove-openinference-external-span-provider branch from a804948 to 88db4ea Compare April 7, 2026 23:04
Copy link
Copy Markdown
Collaborator

@ionmincu ionmincu left a comment

Choose a reason for hiding this comment

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

do not remove them

@cfauchere cfauchere closed this Apr 8, 2026
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