chore: dev to main merge#160
Merged
Merged
Conversation
…ployment (#157) * fix: add support for reusing existing Azure AI Foundry projects in deployment Co-authored-by: Copilot <copilot@github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * fix: improve handling of existing AI project resource ID in Bicep template * fix: update deployment guide to include instructions for reusing existing Azure resources and add new reference documents * fix: add documentation for reusing existing Fabric capacity and workspace in BYO mode --------- Co-authored-by: Copilot <copilot@github.com> Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Co-authored-by: Harmanpreet-Microsoft <v-harmanprka@microsoft.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Adds “bring your own” (BYO) support for reusing an existing Azure AI Foundry project (including cross-subscription), and improves docs by splitting reuse instructions into dedicated guides.
Changes:
- Add BYO AI Foundry project support via
AZURE_EXISTING_AI_PROJECT_RESOURCE_ID(Bicep + preprovision env wiring). - Disable Foundry/account+project creation automatically when BYO project resource ID is provided.
- Add dedicated “reuse existing resources” documentation pages and link them from the deployment guide.
Reviewed changes
Copilot reviewed 8 out of 14 changed files in this pull request and generated 10 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/preprovision-integrated.ps1 | Parses optional existing Foundry project resource ID and sets azd env values to target the BYO project (incl. cross-subscription discovery). |
| infra/main.bicepparam | Reads AZURE_EXISTING_AI_PROJECT_RESOURCE_ID and auto-disables deployAiFoundry / deployAfProject when set. |
| infra/main.bicep | Adds existingAiProjectResourceId parameter, parses it into effective Foundry identifiers, and outputs those for downstream automation. |
| docs/re-use-log-analytics.md | New guide for reusing an existing Log Analytics workspace. |
| docs/re-use-foundry-project.md | New guide for reusing an existing Azure AI Foundry project. |
| docs/re-use-fabric-capacity.md | New guide for reusing an existing Fabric capacity/workspace. |
| docs/parameter_guide.md | Removes embedded BYO sections (moved into dedicated guides). |
| docs/deploymentguide.md | Links to the new dedicated reuse guides and documents the new Foundry BYO parameter. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Prajwal-Microsoft
approved these changes
May 26, 2026
Co-authored-by: Avijit-Microsoft <v-aghorui@microsoft.com>
Avijit-Microsoft
approved these changes
May 26, 2026
|
🎉 This PR is included in version 1.5.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
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 updates the deployment documentation and infrastructure templates to standardize and clarify the process for reusing ("BYO" mode) existing Azure resources, especially Azure AI Foundry projects, Log Analytics workspaces, and Fabric capacities/workspaces. The changes introduce dedicated, step-by-step guides for each BYO scenario, refactor and expand documentation for easier navigation, and enhance infrastructure parameterization and output for BYO support.
Key changes include:
Documentation Restructuring and Clarity
deploymentguide.mdandparameter_guide.md, replacing them with concise links to new, dedicated guides for reusing existing Azure AI Foundry projects, Log Analytics workspaces, and Fabric capacity/workspaces. This makes the main documentation more readable and easier to navigate. [1] [2] [3]re-use-foundry-project.md,re-use-log-analytics.md, andre-use-fabric-capacity.md, each providing step-by-step instructions (with screenshots) for locating resource IDs and configuring environment variables for BYO scenarios. [1] [2] [3]Infrastructure Parameterization and Outputs
existingAiProjectResourceIdparameter toinfra/main.bicepandinfra/main.bicepparam, allowing deployments to target an existing Azure AI Foundry project by providing its ARM resource ID. The template parses this ID and sets relevant outputs for downstream automation. [1] [2] [3]AZURE_EXISTING_AI_PROJECT_RESOURCE_IDis set, ensuring idempotent BYO deployments.deploymentguide.mdto document the newexistingAiProjectResourceIdparameter and its environment variable.These changes improve the clarity, usability, and flexibility of the deployment process for users who wish to reuse existing Azure resources.
References:
Does this introduce a breaking change?
Golden Path Validation
Deployment Validation
What to Check
Verify that the following are valid
Other Information