Skip to content

chore: Dev to Main merge#783

Open
Ragini-Microsoft wants to merge 38 commits intomainfrom
dev
Open

chore: Dev to Main merge#783
Ragini-Microsoft wants to merge 38 commits intomainfrom
dev

Conversation

@Ragini-Microsoft
Copy link
Collaborator

Purpose

This pull request introduces several improvements and refactors across the infrastructure, documentation, and environment configuration for the Content Generation solution accelerator. The main focus is on standardizing parameter names, updating module versions for improved reliability and features, and clarifying documentation for post-deployment and technical guidance. The most important changes are grouped below:

Parameter and Environment Variable Standardization:

  • Refactored parameter and environment variable names for GPT and image models, Cosmos DB, and storage accounts throughout documentation (docs/CustomizingAzdParameters.md, .env.sample, docs/TECHNICAL_GUIDE.md). This ensures consistent naming across deployment, configuration, and code. [1] [2] [3] [4]

Infrastructure Module and Resource Updates:

  • Upgraded AVM Bicep module versions for Log Analytics Workspace, User Assigned Identity, Private DNS Zones, AI Services, Private Endpoint, and AI Search to latest releases, improving stability and feature set (infra/main.bicep). [1] [2] [3] [4] [5]
  • Changed AI Search replica count from 2 to 3 when redundancy is enabled and updated hostingMode value to 'Default' for clarity and correctness (infra/main.bicep).
  • Updated API version for AI Search Foundry connection resource to 2025-09-01 for future compatibility (infra/main.bicep).

Resource Tagging and Telemetry Improvements:

  • Improved resource group tag handling by referencing the existing resource group and updating tags API version, ensuring tags are preserved and managed correctly (infra/main.bicep). [1] [2]
  • Added enableTelemetry parameter propagation to more modules, clarifying its scope and improving telemetry control (infra/main.bicep). [1] [2]

Documentation Enhancements:

  • Added a new comprehensive AVM Post Deployment Guide (docs/AVMPostDeploymentGuide.md) detailing manual post-deployment steps, prerequisites, and verification checks for AVM-based deployments.
  • Clarified technical guide settings and parameter names for easier developer onboarding and troubleshooting (docs/TECHNICAL_GUIDE.md).

Minor Fixes and Improvements:

  • Made acrName parameter optional and clarified telemetry parameter descriptions for improved user experience (infra/main.bicep).
  • Fixed dependency logic for virtual network deployment to avoid unnecessary dependencies when using existing Log Analytics workspace (infra/main.bicep).
  • Corrected data types for Log Analytics workspace quota and improved descriptions for module parameters (infra/main.bicep).

These changes collectively improve deployment reliability, configuration consistency, and user guidance for the solution accelerator.

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.

AjitPadhi-Microsoft and others added 28 commits March 3, 2026 15:45
- Updated parameter references in main.parameters.json and main.waf.parameters.json to use consistent naming conventions.
- Changed API versions for Cognitive Services and Container Instances in Bicep modules to the latest preview versions.
- Enhanced role assignment module to reference updated API versions for existing AI Services and Projects.
- Modified web-sites.bicep to include telemetry options and updated API versions, ensuring compatibility with the latest Azure features.
- Improved post_deploy.py script to fetch deployment outputs from Azure, allowing for dynamic configuration based on existing resources.
chore: main to dev downmerge
fix: Updated azure.yaml file to exclude the 1.23.9 azd version
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.

9 participants