fix: r/vsphere_vm_storage_policy delete method check for use (#1863)

Modify the `resourceVMStoragePolicyDelete` method to check the response of `pbmClient.DeleteProfile()` API call for errors and re-throw them. As a result, if a policy is in use and cannot be deleted, the destroy operation will fail and the policy will remain in the state.

vsphere/resource_vsphere_vm_storage_policy.go

@@ -275,7 +275,7 @@ func resourceVMStoragePolicyUpdate(d *schema.ResourceData, meta interface{}) err
 }
 
 func resourceVMStoragePolicyDelete(d *schema.ResourceData, meta interface{}) error {
-	log.Printf("[DEBUG] Performing create of VM storage policy with ID %s", d.Id())
+	log.Printf("[DEBUG] Performing delete of VM storage policy with ID %s", d.Id())
 	client := meta.(*Client).vimClient
 	pbmClient, err := pbm.NewClient(context.Background(), client.Client)
 	if err != nil {
@@ -286,10 +286,15 @@ func resourceVMStoragePolicyDelete(d *schema.ResourceData, meta interface{}) err
 		UniqueId: d.Id(),
 	})
 
-	_, err = pbmClient.DeleteProfile(context.Background(), policyIdsToDelete)
+	var deleteProfileOutcome []types2.PbmProfileOperationOutcome
+	deleteProfileOutcome, err = pbmClient.DeleteProfile(context.Background(), policyIdsToDelete)
 	if err != nil {
 		return fmt.Errorf("error while deleting policy with ID %s %s", d.Id(), err)
 	}
+	if len(deleteProfileOutcome) > 0 && deleteProfileOutcome[0].Fault != nil {
+		return fmt.Errorf("error while deleting policy with ID %s %s",
+			d.Id(), deleteProfileOutcome[0].Fault.LocalizedMessage)
+	}
 	d.SetId("")
 	log.Printf("[DEBUG] %s: Delete complete", d.Id())
 	return nil