Skip to content

fix: Prevent removal of existing RG tags when deploying GSA#483

Merged
Prajwal-Microsoft merged 1 commit intodevfrom
fix-gsa-deploy
Oct 28, 2025
Merged

fix: Prevent removal of existing RG tags when deploying GSA#483
Prajwal-Microsoft merged 1 commit intodevfrom
fix-gsa-deploy

Conversation

@Kanchan-Microsoft
Copy link
Copy Markdown
Contributor

Purpose

This pull request makes a small but important change to how tags are applied to resources in the infra/main.bicep file. The change ensures that all tags from the resource group are included when setting tags for the resourceGroupTags resource.

  • Tag inheritance improvement:
    • In the resourceGroupTags resource, the spread operator is used to include all tags from the resource group by adding ...resourceGroup().tags to the tags object. This ensures that any tags defined at the resource group level are automatically applied to resources, improving consistency and reducing manual tag duplication.

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

@Kanchan-Microsoft Kanchan-Microsoft changed the title fix: Prevent removal of existing RG tags fix: Prevent removal of existing RG tags when deploying GSA Oct 28, 2025
@Prajwal-Microsoft Prajwal-Microsoft merged commit 27e05ef into dev Oct 28, 2025
6 checks passed
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Nov 3, 2025

🎉 This PR is included in version 1.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@Priyanka-Microsoft Priyanka-Microsoft deleted the fix-gsa-deploy branch December 25, 2025 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants