Skip to content

fix: Update AVM & Bicep modules and resource versions #193

Open
Akhileswara-Microsoft wants to merge 5 commits intodevfrom
AVM_Bicep_Versions_Update_Akhileswara
Open

fix: Update AVM & Bicep modules and resource versions #193
Akhileswara-Microsoft wants to merge 5 commits intodevfrom
AVM_Bicep_Versions_Update_Akhileswara

Conversation

@Akhileswara-Microsoft
Copy link
Copy Markdown

Purpose

This pull request updates the infrastructure as code for the solution accelerator, focusing on modernizing resource definitions, upgrading AVM module versions, and improving compatibility with current Azure best practices. The main changes include updating module versions, resource API versions, and aligning parameters with the latest standards.

These updates collectively ensure the solution remains compatible with the latest Azure platform features and best practices, while improving maintainability and future-proofing the deployment scripts.

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.

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

This PR modernizes the solution’s IaC by upgrading AVM Bicep module versions and Azure resource API versions, aiming to align deployments with newer Azure/AVM schemas and best practices.

Changes:

  • Bump multiple AVM modules (networking, storage, Cosmos DB, Container Apps, Bastion, identity, monitoring) to newer versions.
  • Update several ARM resource API versions (e.g., Log Analytics workspace, Cognitive Services resources, tags).
  • Adjust parameter/property names to match updated module contracts (e.g., availabilityZones, availabilityZone, Cosmos failover/role definition properties, Container Apps logging config).

Reviewed changes

Copilot reviewed 8 out of 10 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
infra/modules/virtualNetwork.bicep Updates AVM module versions for NSG and VNet.
infra/modules/storageAccount.bicep Updates AVM common types + storage account module version.
infra/modules/role.bicep Updates Cognitive Services API version for existing account reference.
infra/modules/cosmosDb.bicep Updates AVM common types, Cosmos role definition API version, and Cosmos AVM module version.
infra/modules/ai-services-deployments.bicep Updates AVM common types and Cognitive Services API versions (account + deployments).
infra/modules/ai-project.bicep Updates Cognitive Services API versions for projects.
infra/main_custom.bicep Upgrades AVM modules and updates several schema-aligned parameters in the “custom” main template.
infra/main.bicep Upgrades AVM modules and updates several schema-aligned parameters in the primary main template.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread infra/main.bicep
Comment thread infra/main_custom.bicep
Comment thread infra/modules/storageAccount.bicep
Comment thread infra/modules/cosmosDb.bicep
Comment thread infra/main.bicep Outdated
Comment thread infra/main_custom.bicep Outdated
…ices

- Updated cognitive services API version from 2025-06-01 to 2025-12-01 in ai-project.bicep, ai-services-deployments.bicep, and role.bicep.
- Updated cognitive services deployments API version from 2024-10-01 to 2025-12-01 in ai-services-deployments.bicep.
- Updated SQL role definitions API version from 2024-11-15 to 2025-10-15 in cosmosDb.bicep.
- Updated role assignment type import version from 0.5.1 to 0.7.0 in ai-services-deployments.bicep, cosmosDb.bicep, and storageAccount.bicep.
- Updated storage account module version from 0.26.2 to 0.32.0 in storageAccount.bicep.
- Updated network security group module version from 0.5.1 to 0.5.3 and virtual network module version from 0.7.0 to 0.8.0 in virtualNetwork.bicep.
…ole assignments in Cosmos DB module; remove delete retention policy from Storage Account module
@Akhileswara-Microsoft
Copy link
Copy Markdown
Author

@copilot resolve the merge conflicts in this pull request

…e versions and add new Cosmos DB container variable

Co-authored-by: Akhileswara-Microsoft <234037885+Akhileswara-Microsoft@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 24, 2026

@copilot resolve the merge conflicts in this pull request

Merge conflicts have been resolved in commit 8dd27f6.

The conflicts were in infra/main.bicep, infra/main_custom.bicep, and infra/main.json. Here's how each was resolved:

infra/main.bicep (3 conflicts):

  • Added the new processControlCosmosContainerName variable from dev, while keeping the updated Cosmos DB module version 0.19.0 from this branch
  • Moved processorContainerAppName variable declaration before containerAppBackend (as introduced in dev), while keeping the newer container app module version 0.22.0
  • Removed the duplicate processorContainerAppName declaration near containerAppProcessor (since it's now declared earlier), keeping version 0.22.0

infra/main_custom.bicep (2 conflicts): Same pattern — moved the processorContainerAppName variable and removed its duplicate, keeping module version 0.22.0.

infra/main.json: Kept our branch's compiled ARM template (which reflects the updated AVM module versions).

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • aka.ms
    • Triggering command: /usr/local/bin/bicep bicep build infra/main.bicep --outfile infra/main.json (dns block)
  • centralus.data.mcr.microsoft.com
    • Triggering command: /usr/local/bin/bicep bicep build infra/main.bicep --outfile infra/main.json (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

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.

3 participants