Skip to content

Fix HTTP tracing suppression in agents GA#41361

Merged
lmolkova merged 1 commit intoAzure:feature/azure-ai-agents/1.0.1from
lmolkova:http-suppressions-agents-ga
Jun 2, 2025
Merged

Fix HTTP tracing suppression in agents GA#41361
lmolkova merged 1 commit intoAzure:feature/azure-ai-agents/1.0.1from
lmolkova:http-suppressions-agents-ga

Conversation

@lmolkova
Copy link

extracted from #41341 - fixing double-http-tracing in agents ga version

Copilot AI review requested due to automatic review settings May 30, 2025 14:57
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes double HTTP tracing in agents for the GA version, while also introducing minor improvements in naming, instrumentation, and asynchronous function tool execution.

  • Update package display name for consistency
  • Replace iterative tool call execution with concurrent calls using asyncio.gather in sample files
  • Update telemetry instrumentation to properly use span context and add tracing for create_thread_and_run
  • Bump version and update CHANGELOG

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
sdk/ai/azure-ai-agents/setup.py Corrected display name capitalization
sdk/ai/azure-ai-agents/samples/agents_async/sample_agents_stream_eventhandler_with_functions_async.py Revised tool call execution to use AsyncToolSet and concurrent execution
sdk/ai/azure-ai-agents/samples/agents_async/sample_agents_functions_async.py Revised tool call execution to use AsyncToolSet and concurrent execution
sdk/ai/azure-ai-agents/azure/ai/agents/telemetry/_ai_agents_instrumentor.py Modified span context usage and added instrumentation for create_thread_and_run
sdk/ai/azure-ai-agents/azure/ai/agents/models/_patch.py Updated asynchronous execution of tool calls with asyncio.gather
sdk/ai/azure-ai-agents/azure/ai/agents/_version.py Bumped version
sdk/ai/azure-ai-agents/CHANGELOG.md Updated changelog with brief release notes
Comments suppressed due to low confidence (2)

sdk/ai/azure-ai-agents/azure/ai/agents/telemetry/_ai_agents_instrumentor.py:1475

  • Consider adding automated tests to verify the telemetry instrumentation for 'AgentsClient.create_thread_and_run' to ensure the new context changes work as expected in both sync and async flows.
if class_function_name.startswith("AgentsClient.create_thread_and_run"):

sdk/ai/azure-ai-agents/samples/agents_async/sample_agents_stream_eventhandler_with_functions_async.py:49

  • Please confirm that the 'add' method on AsyncToolSet is supported and documented, ensuring consistency with its usage in other parts of the SDK.
toolset.add(functions)

@lmolkova lmolkova changed the base branch from main to feature/azure-ai-agents/1.0.1 May 30, 2025 14:58
@lmolkova lmolkova changed the title Fix HTTP tracing suppression in AI SDKs Fix HTTP tracing suppression in agents GA May 30, 2025
@lmolkova lmolkova merged commit 9247ed3 into Azure:feature/azure-ai-agents/1.0.1 Jun 2, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants