Im not sure where the error is in your logic. Maybe you can look at restructuring your method. Below is an excerpt from one of my workflows where I am selecting a datastore for a VM to go onto. The difference is that I am looking at the cluster level. Hope this helps:
var allStorage = vmCluster.datastore;
var vmStorage = new Array();
var storageName;
var byteToGB = 1073741824; //1073741824 = 1024*1024*1024 = 1GB
//Determine size of VM here
var storageSum = calcDisks(currentVM);
System.log("VM Size: " + storageSum);
storageSum += 50; //50GB of headroom on datastore required.
for (ii in allStorage)
{
storageName = allStorage[ii].name
if (storageName.search("-DS") >= 0) //This is specific to my company. I wanted to make sure to weed out any datastores that dont have our std naming scheme
{
vmStorage.push(allStorage[ii]);
}
}
var ii = 0;var vmStorageInfo;
var vmStorageLength = vmStorage.length;
while(vmDatastore == null && ii < vmStorageLength)
{
//extract storageinfo and compare free space to needed space
vmStorageInfo = vmStorage[ii].info;
if ((vmStorageInfo.freeSpace / byteToGB) >= storageSum)
{
vmDatastore = vmStorage[ii];
}
ii++;
}
System.log("Selected VM Datastore: " + vmDatastore.name);
function calcDisks(VM)
{
var total = 0;
var devices = VM.config.hardware.device;
var h = 0;
for (var i in devices) {
if (devices[i] instanceof VcVirtualDisk) {
total += devices[i].capacityInKB / 1024 / 1024; //Produces VM size in GB
}
}
return total;
}
.