Azure / avdaccelerator

AVD Accelerator deployment automation to simplify the setup of AVD (Azure Virtual Desktop) based on best practices
MIT License
305 stars 198 forks source link

remove tailing } from resource id in cm-resource-parent tag #610

Closed mbrat2005 closed 2 months ago

mbrat2005 commented 2 months ago

Overview/Summary

The cm-resource-parent tag is not being populated with a valid resource ID - the subscription ID has as tailing }

This PR fixes/adds/changes/removes

  1. Fixes cm-resource-parent in bicep
  2. Fixes cm-resource-parent in ARM

Breaking Changes

  1. Customers using the incorrect tag value would have to update reporting for future deployments

Testing Evidence

Bicep deployment: image

As part of this Pull Request I have

danycontre commented 2 months ago

@mbrat2005 thanks for your contribution, great findings and fixers.

@swathibhat1 @moisesjgomez