Merge pull request #6947 from hyonholee/errorfix

[Compute] Show target in error message

Author: Maddie Clayton

src/ResourceManager/Compute/Commands.Compute.Test/ScenarioTests/VirtualMachineTests.ps1

@@ -3308,6 +3308,9 @@ function Test-VirtualMachineWriteAcceleratorUpdate
             $st = $vm1 | Update-AzureRmVM -OsDiskWriteAccelerator $true; } `
              "not supported on disks with Write Accelerator enabled";
 
+        $output = $error | Out-String;
+        Assert-True {$output.Contains("Target");}
+
         $st = $vm1 | Update-AzureRmVM -OsDiskWriteAccelerator $false;
     }
     finally

src/ResourceManager/Compute/Commands.Compute/ChangeLog.md

@@ -19,6 +19,7 @@
         - Additional information about change #1
 -->
 ## Current Release
+* Fixed the issue that target is missing in error output.
 * Fixed issue with storage account type for VM with managed disk
 * Fixed issue with default resource groups not being set.
 * Fix AEM Extension cmdlets for other environments, for example Azure China

src/ResourceManager/Compute/Commands.Compute/Common/ComputeCloudException.cs

@@ -57,6 +57,7 @@ protected static string GetErrorMessageWithRequestIdInfo(Rest.Azure.CloudExcepti
                 {
                     sb.AppendLine().AppendFormat("ErrorCode: {0}", errorReturned.Error.Code);
                     sb.AppendLine().AppendFormat("ErrorMessage: {0}", errorReturned.Error.Message);
+                    sb.AppendLine().AppendFormat("ErrorTarget: {0}", errorReturned.Error.Target);
                 }
 
                 if (errorReturned.StartTime != null)