Azure / avdaccelerator

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

Fixed VDOT during Session Host Configuration #698 #709

Closed chbragg closed 1 week ago

chbragg commented 2 weeks ago

Overview/Summary

Replace this with a brief description of what this Pull Request fixes, changes, etc.

This PR fixes/adds/changes/removes

  1. Fixed #698
  2. Excludes Snipping Tool, Calculator, Notepad, and Windows Terminal from getting removed by VDOT.
  3. Changed Optimizations from 'All' to everything except for 'DiskCleanup' because 'DiskCleanup' sometimes causes a permissions error and the deployment will fail.

Breaking Changes

  1. Since VDOT has never run successfully previously, new runs of VDOT may cause changes to apps, services, and policies.

Testing Evidence

image

As part of this Pull Request I have

danycontre commented 1 week ago

@chbragg thank you for your contribution, we have decided to remove VDOT functionality from the code given the variability of what customers could want to be part of the image.

@swathibhat1