Okay, let's clean it up completely and use objects where possible. That way, it's much easier. Here's what I'd do:
$OutputPath = "c:\temp\output.csv"
$VCServers = @()
$VCServer = "" | Select Name, User, Password
$VCServer.Name = "MyVC1"
$VCServer.User = "Admin1"
$VCServer.Password = "Pass1"
$VCServers += $VCServer
$VCServer = "" | Select Name, User, Password
$VCServer.Name = "MyVC2"
$VCServer.User = "Admin2"
$VCServer.Password = "Pass2"
$VCServers += $VCServer
$VCServer = "" | Select Name, User, Password
$VCServer.Name = "MyVC3"
$VCServer.User = "Admin3"
$VCServer.Password = "Pass3"
$VCServers += $VCServer
$MyCol = @()
ForEach ($VC in $VCServers)
{
Connect-VIServer $VC.Name -User $VC.User -Password $VC.Password
ForEach ($VM in Get-VM)
{
$myObj = "" | Select VC, Host, VM, Memory, NumCpu, Description, PowerState, MAC
$myObj.VC = $VC.Name
$myObj.Host = $VM.Host.Name
$myObj.VM = $VM.Name
$myObj.Memory = $VM.Memory
$myObj.NumCpu = $VM.NumCpu
$myObj.Description = $VM.Description
$myObj.PowerState = $VM.PowerState
$myObj.MAC = (($VM | Get-View).Guest.Net | ForEach {$_.MacAddress}
$myCol += $myObj
}
Disconnect-VIServer -Confirm:$False
}
$myCol | Export-Csv $OutputPathI think the MAC thing will not work this way, because if there are multiple nics, this will be an array and the csv will show the type instead of the contents. But give it a go.
Hugo
www.peetersonline.nl
Message was edited by: hugopeeters
Reason: Removed unintended line break in code.
