Skip to content

feat: Foundry SDK changes, FDP changes#461

Merged
Roopan-Microsoft merged 13 commits intodevfrom
PSL-17061-AIFoundry-dev
Jun 18, 2025
Merged

feat: Foundry SDK changes, FDP changes#461
Roopan-Microsoft merged 13 commits intodevfrom
PSL-17061-AIFoundry-dev

Conversation

@Ragini-Microsoft
Copy link
Copy Markdown
Collaborator

@Ragini-Microsoft Ragini-Microsoft commented Jun 6, 2025

Purpose

  • This pull request introduces significant updates to the Azure AI infrastructure and documentation, transitioning from "AI Hub" to "AI Foundry" as the primary framework. Key changes include removing references to the deprecated "AI Hub," adding configurations for "AI Foundry" resources, and updating related documentation and deployment files.

Documentation Updates:

  • Removed AZURE_OPENAI_KEY from required environment variables and added new settings for "AI Foundry" configurations, including AZURE_AI_AGENT_API_VERSION, AZURE_AI_AGENT_ENDPOINT, and AZURE_AI_AGENT_MODEL_DEPLOYMENT_NAME. [1] [2] [3]
  • Introduced USE_AI_FOUNDRY_SDK as an optional flag to enable the AI Foundry SDK.

Infrastructure Transition to AI Foundry:

  • Updated infra/abbreviations.json to replace "AI Hub" abbreviations (aih-, aihp-) with "AI Foundry" abbreviations (aif-, aifp-).
  • Replaced "AI Hub" resources with "AI Foundry" resources in infra/deploy_ai_foundry.bicep, including new definitions for aiFoundry and aiFoundryProject. Removed legacy resources such as storage, aiHub, and aiServices. [1] [2] [3] [4]
  • Updated key vault secrets and outputs to reflect "AI Foundry" endpoints, names, and keys. [1] [2] [3] [4]

App Service Deployment Updates:

  • Removed AzureOpenAIKey parameter and related environment variable settings. Added parameters and environment variables for "AI Foundry," including aiFoundryName, aiFoundryProjectName, and aiFoundryProjectEndpoint. [1] [2] [3]
  • Added USE_AI_FOUNDRY_SDK environment variable to enable the AI Foundry SDK in the app service. [1] [2]
  • Removed legacy Application Insights resource definition.

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • I have built and tested the code locally and in a deployed app
  • For frontend changes, I have pulled the latest code from main, built the frontend, and committed all static files.
  • This is a change for all users of this app. No code or asset is specific to my use case or my organization.

Other Information

@Ragini-Microsoft
Copy link
Copy Markdown
Collaborator Author

@microsoft-github-policy-service agree company="Persistent"

@Ragini-Microsoft Ragini-Microsoft changed the title feat: Added foundry SDK changes feat: Foundry SDK changes, FDP changes Jun 18, 2025
@Pavan-Microsoft Pavan-Microsoft requested a review from Copilot June 18, 2025 05:04

This comment was marked as outdated.

Comment thread scripts/SAMPLE_DATA.md Outdated
@Ragini-Microsoft Ragini-Microsoft requested a review from Copilot June 18, 2025 05:18

This comment was marked as outdated.

@Ragini-Microsoft Ragini-Microsoft requested a review from Copilot June 18, 2025 05:29

This comment was marked as outdated.

@Ragini-Microsoft Ragini-Microsoft force-pushed the PSL-17061-AIFoundry-dev branch from 3878c41 to 4ffbdd6 Compare June 18, 2025 05:37
@Ragini-Microsoft Ragini-Microsoft requested a review from Copilot June 18, 2025 10:28
Copy link
Copy Markdown
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 adds support for Azure AI Foundry alongside Azure OpenAI, updates SDK dependencies and API versions, and extends infrastructure templates to configure Foundry resources.

  • Bump Azure Identity/OpenAI SDK versions and include Foundry-specific packages
  • Introduce _AzureAISettings, init_ai_foundry_client, and use_ai_foundry_sdk toggle
  • Update Bicep modules and .env.sample/documentation for Azure AI Foundry configuration

Reviewed Changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/requirements.txt Updated SDK versions and added azure-ai-projects/azure-ai-inference packages
src/backend/settings.py Added _AzureAISettings, use_ai_foundry_sdk flag and bumped OpenAI preview version
src/app.py Implemented init_ai_foundry_client, toggled Foundry vs OpenAI usage, JSON parsing for titles
src/.env.sample Added AZURE_AI_AGENT_* variables and USE_AI_FOUNDRY_SDK
scripts/SAMPLE_DATA.md Adjusted env variable instructions
infra/scripts/index_scripts/02_process_data.py Switched embeddings to Azure AD auth; removed API key usage
infra/main.bicepparam Updated default OpenAI API version
infra/main.bicep Cleaned up legacy comments, added Foundry resource modules
infra/deploy_app_service.bicep Exposed AI Foundry env vars, removed OpenAI key param
infra/deploy_ai_foundry.bicep Defined AI Foundry account/project resources and role assignments
infra/abbreviations.json Renamed aiHub* keys to aiFoundry*
docs/README_LOCAL.md Documented new Azure AI agent environment settings
Comments suppressed due to low confidence (2)

infra/deploy_app_service.bicep:39

  • [nitpick] The parameter azureOpenaiResource mixes casing in the acronym. Consider renaming to azureOpenAIResource for clearer consistency with other camelCase parameters.
param azureOpenaiResource string = ''

infra/scripts/index_scripts/02_process_data.py:42

  • The variable credential is not defined in this scope. You should instantiate a credential (e.g., credential = DefaultAzureCredential()) before using it in get_bearer_token_provider.
    )

Comment thread src/app.py
Comment thread infra/deploy_app_service.bicep
Comment thread src/app.py
@Roopan-Microsoft Roopan-Microsoft merged commit 5fe69e8 into dev Jun 18, 2025
5 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.4.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants