From 32953a0b2788c40e3d32ee7ce2b4f3b1059fa4bb Mon Sep 17 00:00:00 2001 From: tombuildsstuff Date: Fri, 1 Jun 2018 22:09:39 +0200 Subject: [PATCH] Explaining why we're deliberately ignoring an error --- azurerm/resource_arm_virtual_machine_data_disk_attachment.go | 1 + 1 file changed, 1 insertion(+) diff --git a/azurerm/resource_arm_virtual_machine_data_disk_attachment.go b/azurerm/resource_arm_virtual_machine_data_disk_attachment.go index 1c4138212c5e..04bd3dfdf291 100644 --- a/azurerm/resource_arm_virtual_machine_data_disk_attachment.go +++ b/azurerm/resource_arm_virtual_machine_data_disk_attachment.go @@ -139,6 +139,7 @@ func resourceArmVirtualMachineDataDiskAttachmentCreateUpdate(d *schema.ResourceD // if there's too many disks we get a 409 back with: // `The maximum number of data disks allowed to be attached to a VM of this size is 1.` + // which we're intentionally not wrapping, since the errors good. future, err := client.CreateOrUpdate(ctx, resourceGroup, virtualMachineName, virtualMachine) if err != nil { return fmt.Errorf("Error updating Virtual Machine %q (Resource Group %q) with Disk %q: %+v", virtualMachineName, resourceGroup, name, err)