Skip to content

fix(tracing): set Traces source to CodedAgents#1626

Open
ionmincu wants to merge 1 commit into
mainfrom
fix/traces-source-coded-agents
Open

fix(tracing): set Traces source to CodedAgents#1626
ionmincu wants to merge 1 commit into
mainfrom
fix/traces-source-coded-agents

Conversation

@ionmincu
Copy link
Copy Markdown
Contributor

Summary

  • Change source=Robots to source=CodedAgents in the /api/Traces/spans URL built by LlmOpsHttpExporter
  • Update related tests
  • Bump uipath package version to 2.10.63

Test plan

  • pytest packages/uipath/tests/tracing/test_otel_exporters.py

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added test:uipath-langchain Triggers tests in the uipath-langchain-python repository test:uipath-integrations labels May 12, 2026
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 90%)

See analysis details on SonarQube Cloud

def _build_url(self, span_list: list[Dict[str, Any]]) -> str:
"""Construct the URL for the API request."""
trace_id = str(span_list[0]["TraceId"])
return f"{self.base_url}/api/Traces/spans?traceId={trace_id}&source=Robots"
Copy link
Copy Markdown
Collaborator

@radu-mocanu radu-mocanu May 13, 2026

Choose a reason for hiding this comment

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

are we sure this won t break low code?
we need to double-check that all the spans we emit are dropped in by their runtime

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@saksharthakkar / @JosephMar could validate this change from a tracing perspective

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This also needs to be changed


source exists both inside the model and on the query path (for convenience)

@@ -1,6 +1,6 @@
[project]
name = "uipath"
version = "2.10.62"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

shouldn't we also re-generate the uv.lock?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:uipath-integrations test:uipath-langchain Triggers tests in the uipath-langchain-python repository

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants