Skip to content

Conversation

Abdul-Microsoft
Copy link
Contributor

Purpose

  • This pull request introduces several infrastructure and documentation improvements to streamline Azure deployment for the copilot solution accelerator. The changes focus on modernizing resource provisioning, improving parameter clarity, and enhancing deployment instructions for users.

Infrastructure Modernization:

  • Migrated App Service Plan and Application Insights provisioning to use Azure Verified Modules (AVM) for improved best practices and maintainability in deploy_app_service.bicep. Legacy resource definitions are commented out, and new AVM modules (webServerFarm, applicationInsights) are used instead. [1] [2]
  • Updated resource dependencies in deploy_app_service.bicep to reference new AVM modules instead of legacy resources, ensuring correct deployment order.

Parameter and Resource Definition Improvements:

  • Marked parameters as "Required" and clarified descriptions in deploy_app_service.bicep and deploy_index_scripts.bicep, improving usability and reducing ambiguity during deployment. Some unused parameters are commented out for clarity. [1] [2]
  • Added a new azure.yaml manifest to specify template metadata and required Azure Developer CLI (azd) version, supporting standardized deployment workflows.
  • Introduced a new main.parameters.json file for parameterizing infrastructure deployments, enabling environment-based configuration and easier automation.

Documentation Enhancements:

  • Added a new section to the README.md documenting step-by-step instructions for deploying with Azure Developer CLI (azd), making the deployment process more accessible for users.

Other Changes:

  • Removed the legacy main.bicep file, indicating a shift to the new modular and parameterized infrastructure approach.
  • Updated how the Application Insights instrumentation key is referenced in the web app settings for improved reliability and consistency with AVM outputs.

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

@Roopan-Microsoft Roopan-Microsoft merged commit 75da7f5 into byoc-researcher Sep 29, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants