Skip to content

feat: Merge the AVM WAF and Azd Quota check changes from dev to main#150

Merged
Roopan-Microsoft merged 149 commits intomainfrom
dev
Jun 26, 2025
Merged

feat: Merge the AVM WAF and Azd Quota check changes from dev to main#150
Roopan-Microsoft merged 149 commits intomainfrom
dev

Conversation

@Abdul-Microsoft
Copy link
Copy Markdown
Contributor

Purpose

This pull request introduces significant changes to the deployment process, infrastructure configuration, and documentation for the Azure-based solution. The updates focus on aligning the architecture with Azure's Well-Architected Framework (WAF), simplifying infrastructure definitions, and enhancing deployment options for both sandbox and production environments. Below is a summary of the most important changes:

Deployment and Infrastructure Updates:

  • Updated .github/workflows/deploy.yml to rename the AzureAiServiceLocation parameter to aiDeploymentsLocation for consistency in deployment scripts.
  • Removed legacy Bicep files (infra/deploy_ai_foundry.bicep, infra/deploy_keyvault.bicep, infra/deploy_managed_identity.bicep) to streamline infrastructure definitions. These functionalities are now consolidated into the main deployment process. [1] [2] [3]
  • Added new parameters to infra/main.parameters.json for enhanced configurability, including solutionName, deploymentType, llmModel, and gptModelVersion.

Documentation Enhancements:

  • Expanded the README.md to describe the new WAF-aligned architecture option, linking to detailed documentation and deployment guides.
  • Introduced a new file, docs/ArchitectureWAF.md, detailing the WAF-aligned architecture, its pillars (e.g., security, cost optimization), and deployment configuration.
  • Updated docs/DeploymentGuide.md to include instructions for choosing between sandbox and WAF-aligned deployment options, with guidance on parameter adjustments for production environments. [1] [2]

Configuration Simplifications:

  • Removed the environment and preprovision hooks in azure.yaml, simplifying the deployment configuration by eliminating unused or redundant steps. [1] [2]
  • Consolidated infrastructure parameters in infra/main.bicepparam and removed hardcoded values, improving flexibility for different deployment scenarios.

These changes collectively enhance the scalability, security, and maintainability of the solution while providing clearer guidance for users deploying the architecture in various environments.

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

Seth and others added 30 commits May 27, 2025 15:46
@Roopan-Microsoft Roopan-Microsoft changed the title Feat: Merge the AVM WAF and Azd Quota check changes from dev to main feat: Merge the AVM WAF and Azd Quota check changes from dev to main Jun 26, 2025
@Roopan-Microsoft Roopan-Microsoft merged commit 40b870a into main Jun 26, 2025
9 checks passed
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jul 2, 2025

🎉 This PR is included in version 1.4.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Ritesh-Microsoft pushed a commit that referenced this pull request Oct 10, 2025
feat: Merge the AVM WAF and Azd Quota check changes from dev to main
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