Skip to content

fix: Resolved the tag issue#426

Merged
Prajwal-Microsoft merged 2 commits intodevfrom
psl-tag-issue
Feb 19, 2026
Merged

fix: Resolved the tag issue#426
Prajwal-Microsoft merged 2 commits intodevfrom
psl-tag-issue

Conversation

@Prekshith-Microsoft
Copy link
Contributor

@Prekshith-Microsoft Prekshith-Microsoft commented Feb 18, 2026

This pull request updates the way tags are assigned to the resource group in the infra/main.bicep file. The main improvement is the use of the union function to more robustly combine existing tags, new tags, and default tags, which helps prevent errors when some tag sets are missing or undefined.

Tag assignment improvements:

  • Updated the tags property in the resourceGroupTags resource to use the union function, ensuring that existing resource group tags, additional tags, and default tags are merged safely, even if some are null or undefined.This pull request updates the way tags are merged and applied to the resource group in the infra/main.bicep file. The main improvement is the use of the union function to more robustly combine existing tags, new tags, and additional metadata, ensuring all tags are preserved and merged correctly.

Tag merging improvement:

  • Updated the tags property in the resourceGroupTags resource to use the union function, which safely combines existing resource group tags (handling cases where they might be null), incoming tags, and additional deployment metadata such as TemplateName, Type, CreatedBy, and DeploymentName.## Purpose
  • ...

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

This pull request updates how tags are assigned to the resource group in the infra/main.bicep file. The main improvement is the use of the union function to merge tags more robustly, ensuring that all relevant tags are combined, even if some are undefined.

Tag merging improvement:

  • Replaced the spread operator with the union function to combine resourceGroup().tags, tags, and additional static tags, providing safer and more predictable tag assignment.

@Prajwal-Microsoft Prajwal-Microsoft merged commit 5e40e44 into dev Feb 19, 2026
3 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.

2 participants

Comments