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

Exercise 0: Git Security Change Causes Error on VM Creation #13

Closed feaselkl closed 1 week ago

feaselkl commented 4 months ago

Learners receive an error like "post-checkout hook found during git clone disallowed by default" around CreateWinServerVM when trying to create the Hyper-V VM.

The root cause is a Git security change that recently happened: https://github.com/git-lfs/git-lfs/issues/5749

The VM still gets created but create-vm.ps1 doesn't finish successfully and thus doesn't create the on-prem VM within Hyper-V.

The fix is to manually connect to the Azure VM and run the create-vm.ps1 script. It appears that the VM disk zip file does get downloaded to the repo before the script fails, so all of the assets should be in the C:\git folder.

My recommendation would be to add a note to Exercise 0 indicating that if there is an error in that step, to perform the manual fix. A new version of git lfs may obviate this issue, but in the meantime, deliveries of this training may still run into this problem.

feaselkl commented 1 week ago

This appears to be no longer an issue. The security change was reverted and later corrected, so it will not cause problems for future deliveries.