Skip to content

Fix AgentServer Entra tracing test - #48326

Merged
Shivakishore14 merged 2 commits into
Azure:mainfrom
Shivakishore14:sshiva/fix-agentserver-entra-test
Jul 29, 2026
Merged

Fix AgentServer Entra tracing test#48326
Shivakishore14 merged 2 commits into
Azure:mainfrom
Shivakishore14:sshiva/fix-agentserver-entra-test

Conversation

@Shivakishore14

Copy link
Copy Markdown
Member

Description

Fixes the AgentServer Core Entra tracing tests after _setup_distro_export added the required metric_readers keyword argument.

Validation

  • pytest tests/test_tracing.py::TestEntraAuthMode -q
  • azpysdk whl . from sdk/agentserver/azure-ai-agentserver-core

Add the metric_readers argument required by _setup_distro_export so the Entra auth tests match the current helper signature.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 0c368201-b3c0-4a22-8311-2c288cbd374c
@github-actions github-actions Bot added the Hosted Agents sdk/agentserver/* label Jul 29, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
9 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@Shivakishore14
Shivakishore14 marked this pull request as ready for review July 29, 2026 06:49
Copilot AI review requested due to automatic review settings July 29, 2026 06:49
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
9 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Entra tracing test to match _setup_distro_export’s required signature.

Changes:

  • Passes an empty metric_readers list in the test helper.

@Shivakishore14
Shivakishore14 merged commit 9ace6ea into Azure:main Jul 29, 2026
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Hosted Agents sdk/agentserver/*

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants