Skip to content
This repository has been archived by the owner on Dec 6, 2023. It is now read-only.

Changes to reflect current best practices. #123

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

HenryGelderbloem
Copy link

@HenryGelderbloem HenryGelderbloem commented Mar 4, 2022

I have updated the following to reflect updated best practices:

  1. Schema. The existing schema is deprecated so I have updated to it to the latest version.
  2. VM Size. I have changed the build VM size to use the latest version of the D-Series to match what should be used in production.
  3. SKU. Microsoft have released a SKU specifically for AVD that includes Office, Teams & FSLogix. I have updated the Offer and SKU to ensure that this gets used. Side note, the FSLogix script will require updating as it will fail if used with this Offer and SKU as it is already installed. FSLogix should be downloaded and version compared to the installed version to determine if an update (uninstall then install) is required.
  4. Teams. I have removed this as Teams machine-wide installer is included in the image.
  5. I have updated the FsLogix installer completely to download the latest version from Microsoft and compare the version number with that installed in the image. If it is out of date, the installed version will be uninstalled and the downloaded version is installed.

@HenryGelderbloem HenryGelderbloem changed the title Changes to VM size and SKU reflect current best practices. Changes to reflect current best practices. Mar 4, 2022
@HenryGelderbloem HenryGelderbloem deleted the branch danielsollondon:master March 4, 2022 16:46
@HenryGelderbloem HenryGelderbloem deleted the master branch March 4, 2022 16:46
@HenryGelderbloem HenryGelderbloem restored the master branch March 4, 2022 16:46
@HenryGelderbloem
Copy link
Author

Mistakenly closed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants