Skip to content

Python: Add missing fields to AzureAIAgentSettings#12211

Merged
eavanvalkenburg merged 1 commit intomicrosoft:mainfrom
KanchiShimono:fix-azure-ai-agent-settings
May 21, 2025
Merged

Python: Add missing fields to AzureAIAgentSettings#12211
eavanvalkenburg merged 1 commit intomicrosoft:mainfrom
KanchiShimono:fix-azure-ai-agent-settings

Conversation

@KanchiShimono
Copy link
Contributor

Motivation and Context

Some fields were missing from AzureAIAgentSettings, which caused errors during placeholder replacement in the declarative agent.

field_mapping.update({
"ChatModelId": getattr(settings, "model_deployment_name", None),
"Endpoint": getattr(settings, "endpoint", None),
"AgentId": getattr(settings, "agent_id", None),
"BingConnectionId": getattr(settings, "bing_connection_id", None),
"AzureAISearchConnectionId": getattr(settings, "azure_ai_search_connection_id", None),
"AzureAISearchIndexName": getattr(settings, "azure_ai_search_index_name", None),
})

Description

  • Add the missing fields to AzureAIAgentSettings.

Contribution Checklist

@KanchiShimono KanchiShimono requested a review from a team as a code owner May 21, 2025 08:07
@markwallace-microsoft markwallace-microsoft added the python Pull requests for the Python Semantic Kernel label May 21, 2025
@github-actions github-actions bot changed the title Add missing fields to AzureAIAgentSettings Python: Add missing fields to AzureAIAgentSettings May 21, 2025
Copy link
Collaborator

@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.

Thanks for including @KanchiShimono. These can also passed in via extras dict, but yeah, they should be here, too. Appreciate the support.

@markwallace-microsoft
Copy link
Member

Python Unit Test Overview

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

@eavanvalkenburg eavanvalkenburg added this pull request to the merge queue May 21, 2025
Merged via the queue into microsoft:main with commit 318e092 May 21, 2025
30 checks passed
@KanchiShimono KanchiShimono deleted the fix-azure-ai-agent-settings branch May 21, 2025 10:09
jcruzmot-te pushed a commit to thousandeyes/aia-semantic-kernel that referenced this pull request Sep 15, 2025
### Motivation and Context

Some fields were missing from `AzureAIAgentSettings`, which caused
errors during placeholder replacement in the declarative agent.

https://github.com/microsoft/semantic-kernel/blob/6efcc847c56d742a12dec871bd715319a599533c/python/semantic_kernel/agents/azure_ai/azure_ai_agent.py#L568-L575

### Description

- Add the missing fields to `AzureAIAgentSettings`.

### Contribution Checklist

<!-- Before submitting this PR, please make sure: -->

- [x] The code builds clean without any errors or warnings
- [x] The PR follows the [SK Contribution
Guidelines](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md#development-scripts)
raises no violations
- [x] All unit tests pass, and I have added new tests where possible
- [x] I didn't break anyone 😄
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

python Pull requests for the Python Semantic Kernel

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants