Efidisk cannot parse disk size "528K"

[RedPaes]

Describe the bug
Clone a VM with Uefi & a efidisk fails. It canot parse the size string:

Error: failed to decode HTTP GET response (path: nodes/pmx07/qemu/103/config) - Reason: cannot parse disk size "528K"
Looks like the parser for Disks is missing a "K" option:

terraform-provider-proxmox/proxmox/utils.go

Line 56 in b33a8ce

Didn't have a deep lool... But looks like it's related to: #86

To Reproduce

1. Create a Template VM with an EFI Disk as seen in the screenshot below.
2. Copy the VM with Terraform provider
3. Copy will work, but terraform throws an error: Error: failed to decode HTTP GET response (path: nodes/pmx07/qemu/103/config) - Reason: cannot parse disk size "528K"

resource "proxmox_virtual_environment_vm" "windows_vm" {
  name        = var.vm_config.vm_name
  description = "Managed by Terraform"
  tags        = ["terraform", "win-10"]
  node_name = "${var.vm_config.pm_node}"
  clone {
    vm_id = "${var.vm_config.template_id}"
    retries = 3
  }
  agent {
    enabled = true
  }
  disk {
    datastore_id = var.vm_config.datastore_id
    file_format  = "raw" 
    interface    = "scsi0"
    size         = 80
  }
  disk {
    datastore_id = var.vm_config.datastore_id
    file_format  = "raw" 
    interface    = "efidisk0"
    size         = 1
  }
  cpu {
    cores = var.vm_config.cores
  }
  memory {
    dedicated = var.vm_config.ram_mb
  }


  network_device {
    bridge = var.vm_config.network_device
  }

  operating_system {
    type = "win10"
  }

 #}

Expected behavior
Should be able to parse efidisks.

Screenshots
See Efidisk: 512K

[bpg]

I think this will be fixed in #301

[RedPaes]

Thank you for your quick response.

I just had a look at the PR.
As i understand there is no fix for that specific problem currently:

terraform-provider-proxmox/proxmox/utils_test.go

Line 31 in 1ec47dc

{"error on unknown unit string", strPtr("2048K"), -1, true},

I'm not a go developer so maybe i miss something ;)

[bpg]

Ah, you're right. There was a discussion in #86 (comment) where we decided to not support size < 1Gb, missing this use case.

[bpg]

Well, this actually should be fixed in the new #326 😅