Azure / avdaccelerator

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

Scaling plan creation failing [fixed] #573

Closed ripom closed 6 months ago

ripom commented 6 months ago

Overview/Summary

There was a bug in the code that stopped the scaling plan from being created. It was caused by the role assignment not giving the correct role. This resulted in a permission error when creating the scaling plan. I solved the bug by changing the application_id to object_id, as the terraform documentation suggests.

This PR fixes/adds/changes/removes

  1. I have fixed the error in the 4 scenarios in the greenfield folder

Breaking Changes

  1. Fix the error in the role assignment using object_id instead of application_id

Testing Evidence

I have personally tested only the ADDS scenario, that make me confident the scaling plan creation is working as expected as a consequence of right role assignment fixed.

As part of this Pull Request I have

danycontre commented 6 months ago

@ripom thanks for your contribution.

@jensheerin @sihbher please you help checking.