Adding Space used to current script

spiceuser-fjucg · 2021-03-02T12:17:00.000Z

Good Morning!

Thank you for taking the time to read this question. Currently, this script was found online and works like a charm. I just need it to report everything as it currently does but I want another item to show. I want it to display actual disk usage. Any help would be much appreciated!

#Define how you want

$serverList = "C:\Temp\Reports\ServerList.txt"

#only for ad servers

#$serverlist=Get-ADComputer -Filter  {OperatingSystem -Like '*windows server*' }  | Select-Object -ExpandProperty Name

#Only for ad computers
#or $serverlist=Get-ADComputer -Filter  {OperatingSystem -notLike '*server*' }  | Select-Object -ExpandProperty Name

$outputCSV = "C:\Temp\Reports\RFV.csv"
 
 
$scriptpath = $MyInvocation.MyCommand.Path
#$dir = Split-Path $scriptpath
pushd $dir
 
[System.Collections.ArrayList]$sysCollection = New-Object System.Collections.ArrayList($null)
  
foreach ($server in (Get-Content $serverList))
{
    "Collecting information from $server"
    $totCores=0
  
    try
    {
        [wmi]$user = Get-WmiObject Win32_Computersystem -ComputerName $server 
        [wmi]$sysInfo = get-wmiobject Win32_ComputerSystem -Namespace "root\CIMV2" -ComputerName $server -ErrorAction Stop
        [wmi]$bios = Get-WmiObject Win32_BIOS -Namespace "root\CIMV2" -computername $server
        [wmi]$os = Get-WmiObject Win32_OperatingSystem -Namespace "root\CIMV2" -Computername $server
        [array]$disks = Get-WmiObject Win32_LogicalDisk -Namespace "root\CIMV2" -Filter DriveType=3 -Computername $server
        [array]$disks = Get-WmiObject Win32_LogicalDisk -Namespace "root\CIMV2" -Computername $server
        [array]$procs = Get-WmiObject Win32_Processor -Namespace "root\CIMV2" -Computername $server
        [array]$mem = Get-WmiObject Win32_PhysicalMemory -Namespace "root\CIMV2" -ComputerName $server
        [array]$nic = Get-WmiObject Win32_NetworkAdapterConfiguration -Namespace "root\CIMV2" -ComputerName $server | where{$_.IPEnabled -eq "True"}
  
        $si = @{
            user            =[string]$sysinfo.username
            Server          = [string]$server
            Manufacturer    = [string]$sysInfo.Manufacturer
            Model           = [string]$sysInfo.Model
            TotMem          = "$([string]([System.Math]::Round($sysInfo.TotalPhysicalMemory/1gb,2))) GB"
            BiosDesc        = [string]$bios.Description
            BiosVer         = [string]$bios.SMBIOSBIOSVersion+"."+$bios.SMBIOSMajorVersion+"."+$bios.SMBIOSMinorVersion
            BiosSerial      = [string]$bios.SerialNumber
            OSName          = [string]$os.Name.Substring(0,$os.Name.IndexOf("|") -1)
            ServicePack = (“SP {0}” -f [string]$os.ServicePackMajorVersion)
            Arch            = [string]$os.OSArchitecture
            Processors      = [string]@($procs).count
            Cores           = [string]$procs[0].NumberOfCores

        }
         
        $disks | foreach-object {$si."Drive$($_.Name -replace ':', '')"="$([string]([System.Math]::Round($_.Size/1gb,2)))  GB" }

       
       
    }
    catch [Exception]
    {
        "Error communicating with $server, skipping to next"
        $si = @{
            Server          = [string]$server
            ErrorMessage    = [string]$_.Exception.Message
            ErrorItem       = [string]$_.Exception.ItemName
        }
        Continue
    }
    finally
    {
       [void]$sysCollection.Add((New-Object PSObject -Property $si))   
    }
}
  
$sysCollection `
    | select-object Server,TotMem,OSName,Processors,Cores,Manufacturer,Model,BiosSerial,DriveA,DriveB,DriveC,DriveD,DriveE,DriveF,DriveG,DriveH,DriveI,DriveJ,DriveK,DriveL,DriveM,DriveN,DriveO,DriveP,DriveQ,DriveR,DriveS,DriveT,DriveU,DriveV,DriveW,DriveX,DriveY,DriveZ,ErrorMessage,ErrorItem `
    | sort -Property Server `
    | Export-CSV -path $outputCSV -NoTypeInformation  
    Write-Host "inventory completed check $outputCSV" -backgroundcolor "green"

kenannan · 2021-03-02T14:08:28.000Z

Well you do have the correct WMI-OBJECT. Check out this link and use the Select-Object and that should get you started.

theycallmematty · 2021-03-03T07:20:40.000Z

Took the liberty of implementing the suggestion from @kenannan into your code:

#Define how you want

$serverList = "C:\Temp\Reports\ServerList.txt"

#only for ad servers

#$serverlist=Get-ADComputer -Filter  {OperatingSystem -Like '*windows server*' }  | Select-Object -ExpandProperty Name

#Only for ad computers
#or $serverlist=Get-ADComputer -Filter  {OperatingSystem -notLike '*server*' }  | Select-Object -ExpandProperty Name

$outputCSV = "C:\Temp\Reports\RFV.csv"
 
 
$scriptpath = $MyInvocation.MyCommand.Path
#$dir = Split-Path $scriptpath
pushd $dir
 
[System.Collections.ArrayList]$sysCollection = New-Object System.Collections.ArrayList($null)
  
foreach ($server in (Get-Content $serverList))
{
    "Collecting information from $server"
    $totCores=0
  
    try
    {
        [wmi]$user = Get-WmiObject Win32_Computersystem -ComputerName $server 
        [wmi]$sysInfo = get-wmiobject Win32_ComputerSystem -Namespace "root\CIMV2" -ComputerName $server -ErrorAction Stop
        [wmi]$bios = Get-WmiObject Win32_BIOS -Namespace "root\CIMV2" -computername $server
        [wmi]$os = Get-WmiObject Win32_OperatingSystem -Namespace "root\CIMV2" -Computername $server
        [array]$disks = Get-WmiObject Win32_LogicalDisk -Namespace "root\CIMV2" -Filter DriveType=3 -Computername $server
        [array]$disks = Get-WmiObject Win32_LogicalDisk -Namespace "root\CIMV2" -Computername $server
        [array]$procs = Get-WmiObject Win32_Processor -Namespace "root\CIMV2" -Computername $server
        [array]$mem = Get-WmiObject Win32_PhysicalMemory -Namespace "root\CIMV2" -ComputerName $server
        [array]$nic = Get-WmiObject Win32_NetworkAdapterConfiguration -Namespace "root\CIMV2" -ComputerName $server | where{$_.IPEnabled -eq "True"}
  
        $si = @{
            user            =[string]$sysinfo.username
            Server          = [string]$server
            Manufacturer    = [string]$sysInfo.Manufacturer
            Model           = [string]$sysInfo.Model
            TotMem          = "$([string]([System.Math]::Round($sysInfo.TotalPhysicalMemory/1gb,2))) GB"
            BiosDesc        = [string]$bios.Description
            BiosVer         = [string]$bios.SMBIOSBIOSVersion+"."+$bios.SMBIOSMajorVersion+"."+$bios.SMBIOSMinorVersion
            BiosSerial      = [string]$bios.SerialNumber
            OSName          = [string]$os.Name.Substring(0,$os.Name.IndexOf("|") -1)
            ServicePack = (“SP {0}” -f [string]$os.ServicePackMajorVersion)
            Arch            = [string]$os.OSArchitecture
            Processors      = [string]@($procs).count
            Cores           = [string]$procs[0].NumberOfCores

        }
         
        $disks | foreach-object {$si."Drive$($_.Name -replace ':', '')"="{0:N2} GB / {1:N2} GB ({2:P0} used)" -f ( $_.Freespace / 1gb), ($_.Size/1gb), (1 - ($_.Freespace / $_.Size)) }

       
       
    }
    catch [Exception]
    {
        "Error communicating with $server, skipping to next"
        $si = @{
            Server          = [string]$server
            ErrorMessage    = [string]$_.Exception.Message
            ErrorItem       = [string]$_.Exception.ItemName
        }
        Continue
    }
    finally
    {
       [void]$sysCollection.Add((New-Object PSObject -Property $si))   
    }
}
  
$sysCollection `
    | select-object Server,TotMem,OSName,Processors,Cores,Manufacturer,Model,BiosSerial,DriveA,DriveB,DriveC,DriveD,DriveE,DriveF,DriveG,DriveH,DriveI,DriveJ,DriveK,DriveL,DriveM,DriveN,DriveO,DriveP,DriveQ,DriveR,DriveS,DriveT,DriveU,DriveV,DriveW,DriveX,DriveY,DriveZ,ErrorMessage,ErrorItem `
    | sort -Property Server `
    | Export-CSV -path $outputCSV -NoTypeInformation  
    Write-Host "inventory completed check $outputCSV" -backgroundcolor "green"

For each disk, it outputs:
Space used / Total capacity (% used)

You can tweak it to your needs; I just modified line 55:

$disks | foreach-object {$si."Drive$($_.Name -replace ':', '')"=" [...]

kenannan · 2021-03-03T13:30:05.000Z

Looks good to me @theycallmematty . I may need to try this out myself.

spiceuser-fjucg · 2021-03-03T14:19:04.000Z

You guys are both correct, thank you! I wish it would let me give you both credit for the right answer.