Skip to content

chore: dev to main merge#160

Merged
Avijit-Microsoft merged 2 commits into
mainfrom
dev
May 26, 2026
Merged

chore: dev to main merge#160
Avijit-Microsoft merged 2 commits into
mainfrom
dev

Conversation

@Saswato-Microsoft
Copy link
Copy Markdown
Contributor

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

  • Removed lengthy in-line BYO instructions from deploymentguide.md and parameter_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]
  • Added three new guides: re-use-foundry-project.md, re-use-log-analytics.md, and re-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

  • Added a new existingAiProjectResourceId parameter to infra/main.bicep and infra/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]
  • Automatically disables creation of a new AI Foundry account/project if AZURE_EXISTING_AI_PROJECT_RESOURCE_ID is set, ensuring idempotent BYO deployments.
  • Updated the deployment parameter table in deploymentguide.md to document the new existingAiProjectResourceId parameter 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?

  • 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

  • ...

Other Information

…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>
Copilot AI review requested due to automatic review settings May 25, 2026 10:09
@Saswato-Microsoft Saswato-Microsoft changed the title fix: Add support for reusing existing Azure AI Foundry projects in de… chore: dev to main merge May 25, 2026
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

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.

Comment thread infra/main.bicep
Comment thread docs/re-use-log-analytics.md
Comment thread docs/re-use-log-analytics.md
Comment thread docs/re-use-log-analytics.md
Comment thread docs/re-use-foundry-project.md
Comment thread docs/re-use-foundry-project.md
Comment thread docs/re-use-foundry-project.md
Comment thread docs/re-use-foundry-project.md
Comment thread docs/re-use-foundry-project.md
Comment thread docs/re-use-foundry-project.md
Co-authored-by: Avijit-Microsoft <v-aghorui@microsoft.com>
@Avijit-Microsoft Avijit-Microsoft merged commit c13d354 into main May 26, 2026
3 checks passed
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 1.5.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