Powershell - Reading computer name from file error.

Discussion in 'Scripting & Programming' started by Nelix, Apr 23, 2017.

  1. Nelix
    Honorary Member

    Nelix Gigabyte Poster

    1,416
    3
    82
    Hi All

    It has been many years since frequenting this place, but it is good to know that it is still here.

    I have an issue with a Powershell script I am working on. I'm wanting to check the HD's and RAM on all systems listed in a txt file and output it to a .html file.

    My script appears to work and reports drive capacity, used and free space in GB and %. However, it still produces and error but only for some of the machines even though it outputs the details I need for those systems.

    The code snippet is as follows:

    # Start processing disk space
    foreach($computer in $computers)
    {
    $disks = get-wmiobject Win32_ComputerSystem -Computer $computer -Class Win32_LogicalDisk -Filter "DriveType = 3"

    $computer = $computer.toupper()
    foreach($disk in $disks)
    {
    $deviceID = $disk.DeviceID;
    $volName = $disk.VolumeName;
    [float]$size = $disk.Size;
    [float]$freespace = $disk.FreeSpace;
    $percentFree = [Math]::Round(($freespace / $size) * 100);
    $sizeGB = [Math]::Round($size / 1073741824, 2);
    $freeSpaceGB = [Math]::Round($freespace / 1073741824, 2);
    $usedSpaceGB = $sizeGB - $freeSpaceGB;
    $color = $whiteColor;
    # Start processing RAM
    $RAM = Get-WmiObject -ComputerName $computer -Class Win32_OperatingSystem
    $RAMtotal = $RAM.TotalVisibleMemorySize;
    $RAMAvail = $RAM.FreePhysicalMemory;
    $RAMpercent = [Math]::Round(($RAMavail / $RAMTotal) * 100);


    The error that I get is as follows:

    Get-WmiObject : Cannot validate argument on parameter 'ComputerName'. The argument is null or empty. Provide an argument that is not null or empty, and then try the command again.
    At C:\scripts\DC_Storage_Checks\GetDCSpace.ps1:122 char:38
    + $RAM = Get-WmiObject -ComputerName $computer -Class Win32_OperatingSystem
    + ~~~~~~~~~
    + CategoryInfo : InvalidData: (:) [Get-WmiObject], ParameterBindingValidationException
    + FullyQualifiedErrorId : ParameterArgumentValidationError,Microsoft.PowerShell.Commands.GetWmiObjectCommand


    I get the same error when checking the disks.


    Can anyone shed any light on this?

    Cheers
     
    Certifications: A+, 70-210, 70-290, 70-291, 74-409, 70-410, 70-411, 70-337, 70-347
    WIP: 70-346
  2. SimonD
    Honorary Member

    SimonD Terabyte Poster

    3,681
    440
    199
    Can you please post the entire script because without knowing the script \ line item we won't really be able to help too much.
     
    Certifications: CNA | CNE | CCNA | MCP | MCP+I | MCSE NT4 | MCSA 2003 | Security+ | MCSA:S 2003 | MCSE:S 2003 | MCTS:SCCM 2007 | MCTS:Win 7 | MCITP:EDA7 | MCITP:SA | MCITP:EA | MCTS:Hyper-V | VCP 4 | ITIL v3 Foundation | VCP 5 DCV | VCP 5 Cloud | VCP6 NV | VCP6 DCV | VCAP 5.5 DCA
  3. SimonD
    Honorary Member

    SimonD Terabyte Poster

    3,681
    440
    199
    Actually, + $RAM = Get-WmiObject -ComputerName $computer -Class Win32_OperatingSystem is incorrect, for ram you would want something like -

    $RAM = (Get-WMIObject -class Win32_PhysicalMemory | Measure-Object -Property capacity -Sum | % {[Math]::Round(($_.sum / 1GB),2)})
     
    Certifications: CNA | CNE | CCNA | MCP | MCP+I | MCSE NT4 | MCSA 2003 | Security+ | MCSA:S 2003 | MCSE:S 2003 | MCTS:SCCM 2007 | MCTS:Win 7 | MCITP:EDA7 | MCITP:SA | MCITP:EA | MCTS:Hyper-V | VCP 4 | ITIL v3 Foundation | VCP 5 DCV | VCP 5 Cloud | VCP6 NV | VCP6 DCV | VCAP 5.5 DCA
  4. Nelix
    Honorary Member

    Nelix Gigabyte Poster

    1,416
    3
    82
    I must admit that this is a very old Script and I am very much out of practice when it comes to scripting. Here's the entire thing:

    # Continue even if there are errors
    $ErrorActionPreference = "Continue";

    # Set your warning and critical thresholds
    $percentWarning = 100;
    $percentCritcal = 10;

    # EMAIL PROPERTIES
    # Set the recipients of the report.
    #$users = "[email protected]"

    # REPORT PROPERTIES
    # Path to the report
    $reportPath = "C:\scripts\DC_Storage_Checks\";

    # Report name
    $reportName = "DiskSpaceRpt_$(get-date -format dd-MM-yyyy).html";

    # Path and Report name together
    $diskReport = $reportPath + $reportName

    #Set colors for table cell backgrounds
    $redColor = "#FF0000"
    $orangeColor = "#FBB917"
    $whiteColor = "#FFFFFF"

    # Count if any computers have low disk space. Do not send a report if less than 1.
    $i = 0;

    # Get computer list to check disk space
    $computers = Get-Content "C:\scripts\DC_Storage_Checks\servers.txt";
    $datetime = Get-Date -Format "MM-dd-yyyy_HHmmss";

    # Remove the report if it has already been run today so it does not append to the existing report
    If (Test-Path $diskReport)
    {
    Remove-Item $diskReport
    }
    # Cleanup old files..
    $Daysback = "-7"
    $CurrentDate = Get-Date;
    $DateToDelete = $CurrentDate.AddDays($Daysback);
    Get-ChildItem $reportPath | Where-Object { $_.LastWriteTime -lt $DatetoDelete } | Remove-Item;

    # Create and write HTML Header of report
    $titleDate = get-date -uformat "%m-%d-%Y - %A"
    $header = "
    <html>
    <head>
    <meta http-equiv='Content-Type' content='text/html; charset=iso-8859-1'>
    <title>DiskSpace Report</title>
    <STYLE TYPE='text/css'>
    <!--
    td {
    font-family: Calibri;
    font-size: 12px;
    border-top: 1px solid #999999;
    border-right: 1px solid #999999;
    border-bottom: 1px solid #999999;
    border-left: 1px solid #999999;
    padding-top: 0px;
    padding-right: 0px;
    padding-bottom: 0px;
    padding-left: 0px;
    }
    body {
    margin-left: 5px;
    margin-top: 5px;
    margin-right: 0px;
    margin-bottom: 10px;
    table {
    border: thin solid #000000;
    }
    -->
    </style>
    </head>
    <body>
    <table width='100%'>
    <tr bgcolor='#548DD4'>
    <td colspan='7' height='30' align='center'>
    <font face='calibri' color='#003399' size='4'><strong>Daily Morning Report for $titledate</strong></font>
    </td>
    </tr>
    </table>
    "
    Add-Content $diskReport $header

    # Create and write Table header for report
    $tableHeader = "
    <table width='100%'><tbody>
    <tr bgcolor=#548DD4>
    <td width='10%' align='center'>Server</td>
    <td width='5%' align='center'>Drive</td>
    <td width='15%' align='center'>Drive Label</td>
    <td width='10%' align='center'>Total Capacity(GB)</td>
    <td width='10%' align='center'>Used Capacity(GB)</td>
    <td width='10%' align='center'>Free Space(GB)</td>
    <td width='5%' align='center'>Freespace %</td>
    <td width='5%' align='center'>RAM %</td>
    <td width='5%' align='center'>CPU %</td>
    </tr>
    "
    Add-Content $diskReport $tableHeader

    # Start processing disk space
    foreach($computer in $computers)
    {
    $disks = Get-WmiObject -ComputerName "$computer" -Class Win32_LogicalDisk -Filter "DriveType = 3"
    $computer = $computer.toupper()
    foreach($disk in $disks)
    {
    $deviceID = $disk.DeviceID;
    $volName = $disk.VolumeName;
    [float]$size = $disk.Size;
    [float]$freespace = $disk.FreeSpace;
    $percentFree = [Math]::Round(($freespace / $size) * 100);
    $sizeGB = [Math]::Round($size / 1073741824, 2);
    $freeSpaceGB = [Math]::Round($freespace / 1073741824, 2);
    $usedSpaceGB = $sizeGB - $freeSpaceGB;
    $color = $whiteColor;
    # Start processing RAM
    $RAM = Get-WmiObject -ComputerName $computer -Class Win32_OperatingSystem
    $RAMtotal = $RAM.TotalVisibleMemorySize;
    $RAMAvail = $RAM.FreePhysicalMemory;
    $RAMpercent = [Math]::Round(($RAMavail / $RAMTotal) * 100);

    # Set background color to Orange if just a warning
    if($percentFree -lt $percentWarning)
    {
    $color = $orangeColor

    # Set background color to Red if space is Critical
    if($percentFree -lt $percentCritcal)
    {
    $color = $redColor
    }

    # Create table data rows
    $dataRow = "
    <tr>
    <td width='10%'>$computer</td>
    <td width='5%' align='center'>$deviceID</td>
    <td width='10%' >$volName</td>
    <td width='10%' align='center'>$sizeGB</td>
    <td width='10%' align='center'>$usedSpaceGB</td>
    <td width='10%' align='center'>$freeSpaceGB</td>
    <td width='5%' bgcolor=`'$color`' align='center'>$percentFree</td>
    <td width='5%' align='center'>$RAMpercent</td>
    <td width='5%' align='center'>$CPUpercent</td>
    </tr>
    "
    Add-Content $diskReport $dataRow;
    Write-Host -ForegroundColor DarkYellow "$computer $deviceID percentage free space = $percentFree";
    $i++
    }
    }
    }

    # Create table at end of report showing legend of colors for the critical and warning
    $tableDescription = "
    </table><br><table width='20%'>
    <tr bgcolor='White'>
    <td width='10%' align='center' bgcolor='#FBB917'>No Warning</td>
    <td width='10%' align='center' bgcolor='#FF0000'>Critical less than 10% free space</td>
    </tr>
    "
    Add-Content $diskReport $tableDescription
    Add-Content $diskReport "</body></html>"

    # Send Notification if alert $i is greater then 0
    #if ($i -gt 0)
    #{
    # foreach ($user in $users)
    #{
    # Write-Host "Sending Email notification to $user"
    #
    # $smtpServer = "SERVER.DOMAIN"
    # $smtp = New-Object Net.Mail.SmtpClient($smtpServer)
    # $msg = New-Object Net.Mail.MailMessage
    # $msg.To.Add($user)
    # $msg.From = "[email protected]"
    # $msg.Subject = "DiskSpace Report for $titledate"
    # $msg.IsBodyHTML = $true
    # $msg.Body = get-content $diskReport
    # $smtp.Send($msg)
    # $body = ""
    # }
    # }
     
    Certifications: A+, 70-210, 70-290, 70-291, 74-409, 70-410, 70-411, 70-337, 70-347
    WIP: 70-346

Share This Page

Loading...
  1. This site uses cookies to help personalise content, tailor your experience and to keep you logged in if you register.
    By continuing to use this site, you are consenting to our use of cookies.