Find computers in domain with static IPs - PowerShell

What am I doing wrong?

The backtick ` is the line continuation character, and the first error you get 'missing expression after unary operator' appears to be PowerShell not seeing the -Query line as part of the previous line.

Possibly you are missing the ` at the end of the previous line in your copy of the script, or have got it replaced with an apostrophe by some quirk of copy/paste/font/editor code mangling, or something.

or what should I add/remove from the script to make it run?

Remove most of it, it's convoluted and ugly and redundant for what you want.

Why is it querying for Win32_NetworkAdapterConfiguration things and calling them ints? That's so misleading.

Why is it taking the IPAddress, calling it addr, then calling it ip, then calling it IP?

Why is it taking the GatewayAddress, calling it gw_addr, then gw, then Gateway?

The test for -eq $null seems needless, if you try to use foreach on an empty sequence it will work as expected (skip over it).

Using regexes to pick up IPv4 addresses instead of IPv6 addresses - useful, but waffly. IPv6 addresses use : to delimit them, and IPv4 addresses use ., so if you care for only IPv4 addresses, just look for a . in the address.

Why copy the address out, build a list of results, then select everything out of the list, when you could just print the address as you find it?

Do you even care at all about the gateway address?

Why test if the IP address is empty when you are in a block that's processing the IP address at this point so it can't be null at this point?

Why is the whole thing a chain using get-computers % { code } | select results, when that's no help to you?

I've pruned it back a bit and renamed things and got to the following clearer, simpler structure:

1. Setup.
2. Get the computers
3. For each computer, get the network interfaces with static addresses.
4. For each interface, get the IP addresses.
5. For each IP address, identify the IPv4 ones by looking for .

6. Display the computer name and this IPv4 address.

   param ( 
   [string]$LDAPFilter = '(name=*)'
   )
   
   $wmiQuery = "select IPAddress, DefaultIPGateway from Win32_NetworkAdapterConfiguration where IPEnabled=TRUE and DHCPEnabled=FALSE"
   
   $computers = (Get-ADComputer -LDAPFilter $LDAPFilter)
   foreach ($computer in $computers) { 
   
       $networkAdapters = (Get-WmiObject -ErrorAction SilentlyContinue -ComputerName $computer.DNSHostName -Query $wmiQuery) 
       foreach ($networkAdapter in $networkAdapters) { 
           foreach ($ip in $networkAdapter.IPAddress) 
           { 
               if ($ip -match "\.") 
               { 
                   Write-Host $($computer.DNSHostName), $ip } 
               }
       } 
   }
   

I haven't tested this completely, but it works against one computer to show just the name and static IPs.

N.B. it won't show you any names of computers that it can't contact, but then nor would the original one.

You need to be careful of how the web page cuts your code up. Here's my modified version which works by OU in case you don't want every single computer tested and has a little more information as it runs.

############################################################################ 
## 
##  Script, that returns list of computers with first statically configured 
##  IP Address and Default Gateway. 
##  Requires Active Directory Module 
## 
##  Version: 1.0 
##  Author: [email protected] 
##  Date: 07#23#2012 
## 
############################################################################ 

# %Debug Arguments%='"Value1" "-Verbose"'

param ( [string]$LDAPFilter = '(name=*)', [switch]$Verbose ) 

import-module ActiveDirectory

# $LDAPFilter = '(name=*)'
# $Verbose = $True

if ($Verbose.IsPresent) 
    {
    $VerboseModeString = "enabled."
    $VerboseModeVar = $True
    }
Else
    {
    $VerboseModeString = "not enabled."
    $VerboseModeVar = $False
    }

# $VerboseModeVar = $True

Write-Host "Verbose mode is $VerboseModeString" -ForegroundColor Magenta


# LDAP Paths:
$Singapore_OU =     "OU=Singapore,OU=APAC,OU=TopOU,DC=MyDomain,DC=com"
$BC_OU =        "OU=BC,OU=Australia,OU=APAC,OU=TopOU,DC=MyDomain,DC=com"
$MAL_OF_OU =        "OU=KL Office,OU=Malaysia,OU=APAC,OU=TopOU,DC=MyDomain,DC=com"
$HK_OU =    "OU=HK Warehouse,OU=Hong Kong,OU=APAC,OU=TopOU,DC=MyDomain,DC=com"

Get-ADComputer -SearchBase $HK_OU -LDAPFilter $LDAPFilter | % `
{  
    $name = $_.DNSHostName; 

    if ($VerboseModeVar) 
      { Write-Host -ForegroundColor Yellow "Connecting to $name..." }  
    $ints = Get-WmiObject -ErrorAction SilentlyContinue -ComputerName $name -Query "select IPAddress, DefaultIPGateway from Win32_NetworkAdapterConfiguration where IPEnabled=TRUE and DHCPEnabled=FALSE"; 
    if ($ints -ne $null) 
        { 
            foreach ($int in $ints) 
            { 
                foreach ($addr in $int.IPAddress) 
                { 
                    $ip = $null 
                    if ($addr -match "[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}") 
                    { 
                        $ip = $addr 
                        $gw = $null 
                        foreach ($gw_addr in $int.DefaultIPGateway) 
                        { 
                            if ($gw_addr -match "[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}") 
                            { 
                                $gw = $gw_addr 
                                break 
                            } 
                        } 
                        if ($ip -ne $null) 
                        { 
                            $info = New-Object PSObject -Property `
                            @{ 
                                Name = $name 
                                IP = $ip 
                                Gateway = $gw 
                            } 
                            $info 
                            if ($Verbose.IsPresent) 
                                { Write-Host -ForegroundColor Yellow $info } 
                        } 
                    } 
                } 
            } 
        }
    else
        {
        if ($Verbose.IsPresent) 
            {
            Write-Host "--> Could not contact the computer $Name" -ForegroundColor Red
            }
        }

} | 
Select-Object Name, IP, Gateway

I've used this in my project to rebuild AD domain controllers in order to find static IPs which needs DNS changes and it works 100%!