Fixed a bug for Remove-AzVmssDataDisk in one data disk case (#14553)

grizzlytheodore · web-flow · commit d023518d793a · 2021-03-17T10:41:08.000+08:00
* bug fix

* changelog edit
diff --git a/src/Compute/Compute/ChangeLog.md b/src/Compute/Compute/ChangeLog.md
@@ -20,6 +20,7 @@
 
 -->
 ## Upcoming Release
+* Fixed a bug when 1 data disk attached to VMSS for Remove-AzVmssDataDisk [#13368]
 
 ## Version 4.10.0
 * Added parameter `-EnableHotpatching` to the `Set-AzVMOperatingSystem` cmdlet for Windows machines. 
diff --git a/src/Compute/Compute/Generated/VirtualMachineScaleSet/Config/RemoveAzureRmVmssDataDiskCommand.cs b/src/Compute/Compute/Generated/VirtualMachineScaleSet/Config/RemoveAzureRmVmssDataDiskCommand.cs
@@ -88,10 +88,6 @@ protected override void ProcessRecord()
                 this.VirtualMachineScaleSet.VirtualMachineProfile.StorageProfile.DataDisks.Remove(vDataDisks);
             }
 
-            if (this.VirtualMachineScaleSet.VirtualMachineProfile.StorageProfile.DataDisks.Count == 0)
-            {
-                this.VirtualMachineScaleSet.VirtualMachineProfile.StorageProfile.DataDisks = null;
-            }
             WriteObject(this.VirtualMachineScaleSet);
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -88,10 +88,6 @@ protected override void ProcessRecord()`
`88`	`88`	`this.VirtualMachineScaleSet.VirtualMachineProfile.StorageProfile.DataDisks.Remove(vDataDisks);`
`89`	`89`	`}`
`90`	`90`
`91`		`- if (this.VirtualMachineScaleSet.VirtualMachineProfile.StorageProfile.DataDisks.Count == 0)`
`92`		`- {`
`93`		`- this.VirtualMachineScaleSet.VirtualMachineProfile.StorageProfile.DataDisks = null;`
`94`		`- }`
`95`	`91`	`WriteObject(this.VirtualMachineScaleSet);`
`96`	`92`	`}`
`97`	`93`	`}`