Skip to content

Comments

chore: dev changes to main#431

Merged
Avijit-Microsoft merged 5 commits intomainfrom
dev
Feb 23, 2026
Merged

chore: dev changes to main#431
Avijit-Microsoft merged 5 commits intomainfrom
dev

Conversation

@Roopan-Microsoft
Copy link
Contributor

@Roopan-Microsoft Roopan-Microsoft commented Feb 23, 2026

Purpose

  • This pull request updates the infrastructure deployment templates to improve how resource group tags are merged and to align with the latest Bicep tooling. The most significant changes enhance tag handling logic for resource groups, update the Bicep compiler version, and correct resource scoping for certain Azure resources.

Tag handling improvements:

  • Changed the tag merging logic in both infra/main.bicep and infra/main.json to use the union function instead of shallowMerge, ensuring that resource group tags, user-provided tags, and standard deployment tags are all combined correctly and consistently. This prevents accidental overwrites and ensures all relevant tags are preserved. [1] [2] [3]

Resource scoping fixes:

  • Updated the scope property for Cognitive Services resources (locks, diagnostic settings, role assignments) to use the resourceId function instead of string formatting, ensuring more robust and correct resource referencing. [1] [2] [3] [4] [5] [6]

Bicep tooling and metadata updates:

  • Upgraded the Bicep compiler version from 0.39.26.7824 to 0.40.2.10011 throughout the generated ARM templates, resulting in updated templateHash values and ensuring compatibility with the latest Bicep features and fixes. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12]

Minor corrections and ordering:

  • Fixed the ordering of private DNS zone dependencies in the resources list to match expected deployment order.

These changes collectively improve the reliability, maintainability, and correctness of the infrastructure deployment process.

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

@Avijit-Microsoft Avijit-Microsoft merged commit 48c4c8e into main Feb 23, 2026
8 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