Hi Adam,

Can you add your thoughts here?

Thanks,
Tina

The result here is the return from waitForTask (https://github.com/ManageIQ/vmware_web_service/blob/master/lib/VMwareWebService/MiqVimInventory.rb, https://github.com/ManageIQ/vmware_web_service/blob/master/lib/VMwareWebService/MiqVimInventory.rb#L2240)

Since vm.add_disk is a common vm method we can't return the scsi controller/unit number because not all providers do this (e.g. cloud providers have no concept of this).

We might be able to return the location or filename, something that you could use to lookup the new disk.  We'll need to see what we have available at the time the method returns.

It would have to be a new method, since we can't change the return signature of an existing method.