search

microsoft / TechExcel-Securely-migrate-Windows-Server-and-SQL-Server-workloads-to-Azure

To gain access, please finish setting up this repository now at:
https://repos.opensource.microsoft.com/microsoft/wizard?existingreponame=TechExcel-Securely-migrate-Windows-Server-and-SQL-Server-workloads-to-Azure&existingrepoid=745660430
MIT License
5 stars 16 forks source link

Bicep Deployment Script Notes #15

Open feaselkl opened 4 months ago

feaselkl commented 4 months ago

There are a few issues learners typically run into regarding the Bicep deployment script.

  1. Storage account names need to be unique. It would be a good thing to update the solution to indicate that learners should change the "tailspin" prefix to something unique, noting that the prefix should be no more than 8 characters to prevent errors around storage account names being too long.
  2. The region is hard-coded to North Central US, regardless of the resource group location. That's not necessarily a problem, though there are capacity limits for the Standard_D4s_v5 virtual machine. Several learners per delivery tend to get an error around capacity limits, so it would be a good idea either to use a broader VM type (maybe Standard_D4_v5?) or allow custom regions in the script.
jasonhorner commented 2 weeks ago

it is also worth noting that the deployment ARM also requires the UPN (username) in addition to the user id normally this is the same as the users email address however this may not always be the case.