microsoft / PartsUnlimited

.Net Core + SQL Azure app for DevOps Scenarios
https://microsoft.github.io/PartsUnlimited/
MIT License
667 stars 1.53k forks source link

Please correct Provision-lab-textual-workflow #148

Closed dmi3mis closed 6 years ago

dmi3mis commented 6 years ago

Please remove string number 116 that states as

$vm2 = Set-AzureRmVMOSDisk -VM $vm2 -Name $using:vm2osDiskName -StorageAccountType StandardLRS -DiskSizeInGB $using:vmosDiskSize -CreateOption fromImage -Caching ReadWrite

We have Set-AzureRmVMOSDisk executed 2 times for second vm. First VM creates successfully without error, but when we creating second vm script generates an error. If string 116 exist, script complete with error " New-AzureRmVM : Parameter 'osDisk.managedDisk' is not allowed." Just remove this string and script will execute without error.

eamkel commented 6 years ago

Thanks, tested the change and verified it runs fine, merged the PR.

Thanks for posting! Eamonn