fix: Add support for reusing existing Azure AI Foundry projects in deployment#157
Open
Saswato-Microsoft wants to merge 5 commits into
Open
fix: Add support for reusing existing Azure AI Foundry projects in deployment#157Saswato-Microsoft wants to merge 5 commits into
Saswato-Microsoft wants to merge 5 commits into
Conversation
…ployment Co-authored-by: Copilot <copilot@github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Adds “Bring Your Own” (BYO) Azure AI Foundry project support so deployments can target an existing Foundry account/project (including cross-subscription), instead of always provisioning new Foundry resources.
Changes:
- Introduces
existingAiProjectResourceIdinput and outputs parsed Foundry account/project/RG/subscription details from that resource ID. - Updates preprovisioning to parse
AZURE_EXISTING_AI_PROJECT_RESOURCE_IDand publish the correctazd envvalues for downstream automation. - Expands deployment docs/parameter guide with BYO AI Foundry usage and output references.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/preprovision-integrated.ps1 | Detects/parses an existing AI Foundry project resource ID and sets azd env values for cross-RG/sub usage. |
| infra/main.bicepparam | Reads AZURE_EXISTING_AI_PROJECT_RESOURCE_ID, passes it to main.bicep, and conditionally disables Foundry/project deployment toggles. |
| infra/main.bicep | Adds existingAiProjectResourceId param, parses segments into outputs, and emits BYO-related outputs. |
| docs/parameter_guide.md | Documents BYO AI Foundry behavior, setup, permissions, and outputs. |
| docs/deploymentguide.md | Adds BYO AI Foundry parameter guidance and example azd env set usage. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
…ting Azure resources and add new reference documents
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 7 out of 13 changed files in this pull request and generated 3 comments.
Comments suppressed due to low confidence (1)
docs/parameter_guide.md:92
- The Parameter Guide no longer documents how to configure BYO Log Analytics or the new BYO Foundry project (
existingAiProjectResourceId/AZURE_EXISTING_AI_PROJECT_RESOURCE_ID). This conflicts with the PR description and leaves readers without a reference in this guide; either add the missing section(s) or update the PR description and include a link here to the new reuse guides.
---
## Table of Contents
1. [Basic Parameters](#basic-parameters)
2. [Deployment Toggles](#deployment-toggles)
3. [Network Configuration](#network-configuration)
4. [Microsoft Foundry Configuration](#microsoft-foundry-configuration)
5. [Individual Service Configuration](#individual-service-configuration)
6. [Common Customization Examples](#common-customization-examples)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose
This pull request introduces improvements to the deployment documentation and infrastructure configuration, focusing on making "bring your own" (BYO) resource scenarios clearer and easier to use. The main changes include adding dedicated guides for reusing existing Azure resources (such as Log Analytics, AI Foundry Projects, and Fabric Capacity/Workspace), updating parameter handling in the Bicep templates to support BYO AI Foundry Projects, and simplifying the main deployment guide to reference these new guides.
Documentation Restructuring and Additions:
docs/re-use-foundry-project.md: Step-by-step instructions for reusing an existing Azure AI Foundry Project.docs/re-use-log-analytics.md: Step-by-step instructions for reusing an existing Log Analytics Workspace.docs/re-use-fabric-capacity.md: Step-by-step instructions for reusing an existing Fabric Capacity and Workspace.docs/deploymentguide.md) by removing lengthy, embedded BYO instructions and replacing them with links to the new dedicated guides. [1] [2]docs/parameter_guide.md, which is now covered by the new guides.Infrastructure as Code (Bicep) Enhancements:
existingAiProjectResourceIdparameter ininfra/main.bicepandinfra/main.bicepparam, including logic to parse and use this resource if provided. [1] [2]infra/main.bicepto expose effective AI Foundry resource details and BYO status.deployAiFoundryand related parameters when BYO Foundry Project is detected.existingAiProjectResourceIdparameter in the deployment guide's parameter table.These changes improve maintainability, clarity, and user experience for teams deploying with existing Azure resources.
Does this introduce a breaking change?
Golden Path Validation
Deployment Validation
What to Check
Verify that the following are valid
Other Information