Skip to content

Python: Update the AzureAIAgent packages, types, and APIs based on Azure SDK's new code #12084

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 20 commits into
base: main
Choose a base branch
from
Open

Python: Update the AzureAIAgent packages, types, and APIs based on Azure SDK's new code #12084

wants to merge 20 commits into from
+779 −782

Conversation

moonbox3
Copy link
Contributor

Motivation and Context

The Azure AI Agent Service is moving to GA soon. As part of that, they've made several changes in the underlying APIs, which require the use of new packages, updated type imports and some new underlying APIs. This PR updates the AzureAIAgent to use the latest Azure SDK's new code.

  • The client's connection string has been replaced with an endpoint.
  • The AzureAIAgentSettings's model_deployment_name was updated to deployment_name.
  • Tools are now imported from from azure.ai.agents.models instead of from azure.ai.projects.models.
  • Declarative spec handling for AzureAIAgent was updated based on the new APIs.
  • Unit tests were updated.

Description

Update the AzureAIAgent related code, samples and tests based on the Azure SDK's latest APIs.

Contribution Checklist

@moonbox3 moonbox3 self-assigned this May 15, 2025
@moonbox3 moonbox3 requested a review from a team as a code owner May 15, 2025 11:07
@moonbox3 moonbox3 added PR: breaking change Pull requests that introduce breaking changes agents labels May 15, 2025
@markwallace-microsoft markwallace-microsoft added python Pull requests for the Python Semantic Kernel documentation labels May 15, 2025
@markwallace-microsoft
Copy link
Member

markwallace-microsoft commented May 15, 2025 •
edited
Loading

Python Unit Test Overview

Tests Skipped Failures Errors Time
3584 18 💤 0 ❌ 0 🔥 1m 51s ⏱️

@moonbox3 moonbox3 added this pull request to the merge queue May 16, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 16, 2025
moonbox3
moonbox3 commented May 16, 2025
View reviewed changes
Copy link
Contributor Author

@moonbox3 moonbox3 left a comment

Choose a reason for hiding this comment

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

Waiting for a response from the Azure SDK team about some updates related to this. Holding off for now.

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

@TaoChenOSU TaoChenOSU TaoChenOSU approved these changes

@alliscode alliscode alliscode approved these changes

@crickman crickman crickman approved these changes

At least 2 approving reviews are required to merge this pull request.

Assignees

@moonbox3 moonbox3

Labels
agents documentation PR: breaking change Pull requests that introduce breaking changes python Pull requests for the Python Semantic Kernel
Projects
Semantic Kernel
Status: No status
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@moonbox3 @markwallace-microsoft @TaoChenOSU @alliscode @crickman