Problem with my script, dont execute

Nervill_mcbo Hello. 

I've adapted the script to only check for the Network adapter "Ethernet 3" as you are sure that your VPN connection is using that one. that should fix that for you. 

For the warning I've added a piece that will send an email whenever there is a host down, please update the script with your desired paths and email server/user data. remember that the first time you have to run the script and provide the credentials that will be saved to an XML file, after that you can use the task scheduler and the script will no ask for credentials as it already has the file. 

#Start-Transcript -Path "C:\transcriptPing.txt" -Force

#Add-Type -AssemblyName PresentationCore,PresentationFramework

$names        = @('192.168.50.1','192.168.50.2','8.8.8.8')#Get-content "C:\Servers.txt" #
$subname      = $null
$chequea      = "no"
$dia          = Get-date
#$ButtonType   = [System.Windows.MessageBoxButton]::OK
#$MessageIcon  = [System.Windows.MessageBoxImage]::Error
#$MessageTitle = "Falla de Alcance por Ping"

$ipsV4 = $(Get-NetAdapter -Physical | where{$_.Name -eq 'Ethernet 3'} | Get-NetIPAddress | where{$_.AddressFamily -eq 'IPv4'}).IPAddress 

Foreach($ipV4_tmp in $ipsV4) #added due that multiple Adapter could be up
 {
  $ipV4 = $null
  [System.Net.IPAddress]$ipV4 = $ipV4_tmp
  Write-Host "Local Adapter ip" $ipV4.IPAddressToString -ForegroundColor Cyan

  If($ipV4_tmp)
   {
    if ($ipV4.GetAddressBytes()[0] -eq "192" -and $ipV4.GetAddressBytes()[1]  -eq "168" -and $ipV4.GetAddressBytes()[2] -eq "5" ){
    $chequea="si"
    }
    
    if ($ipV4.GetAddressBytes()[0] -eq "192" -and $ipV4.GetAddressBytes()[1] -eq "168" -and $ipV4.GetAddressBytes()[2] -eq "6" ){
    $chequea="si"
    }
    
    if ($ipV4.GetAddressBytes()[0] -eq "10" -and $ipV4.GetAddressBytes()[1] -eq "212" -and $ipV4.GetAddressBytes()[2] -eq "134" ){
    $chequea="si"
    }
   }
  Else
   {
    Write-Host "Adapter IP item was empty"
   }
 } 

if($chequea -eq "si")
 {
   $Unreachable = @()
  $Output += "$dia : "
  foreach ($name in $names)
   {
    if(Test-Connection -ComputerName $name -Count 1 -ErrorAction SilentlyContinue)
     {
    $Output+= "$name ====>> UP || "
    Write-Host "$name ====>> UP "
   }
    else
     {
    $Output+= "$name ====>> DOWN || "
    Write-Host "$name ====>> DOWN "
    $Unreachable += $name 
    #region Conditions
   if($name -eq "8.8.8.8")
    {
     $subname="DNS Google"
    }
   if($name -eq "www.google.com")
    {
     $subname = "Dominio Google"
    }
   if($name -eq "192.168.5.1")
    {
     $subname="Firewall"
    }
   if($name -eq "192.168.5.2")
    {
     $subname="Server 10"
    }
   if($name -eq "192.168.5.30")
    {
     $subname="Server 8"
    }
   if($name -eq "192.168.5.29")
    {
     $subname="Server 9"
    }
   if($name -eq "192.168.5.52")
    {
     $subname="Server 7"
    }
   if($name -eq "192.168.6.16")
    {
     $subname="Server 1"
    }
   if($name -eq "192.168.5.94")
    {
     $subname="Server 2"
    }
   if($name -eq "192.168.5.102")
    {
     $subname="Server 3" 
    }
   if($name -eq "192.168.5.43")
    {
     $subname="Server 4"
    }
   if($name -eq "192.168.5.124")
    {
     $subname="Server 5"
    }
   if($name -eq "192.168.6.250")
    {
    $subname="Impresora Wifi"
    }
   if($name -eq "192.168.5.250")
    {
     $subname="Impresora Xerox "
    }
   #endregion
    #$MessageBody = "Conexion Perdida con el IP: $name, $subname"
    #$oReturn=[System.Windows.MessageBox]::Show($MessageBody,$MessageTitle,$ButtonType,$MessageIcon)
    
    }#If
   }#foreach
   $Output | Out-file "C:\PingResult.txt" -Append
 }#If

$WorkingDirectory = "C:\"
$credPath = Join-Path -path ($WorkingDirectory) -ChildPath Mail.cred 
$mailCreds = If(Test-Path $credPath)
              {
               Import-Clixml $credPath
              }
             else #Creates Creds File
              {
               $tmpCred = Get-Credential #Provide Mail Creds, you may require a service account 
               $tmpCred | Export-CliXml -Path $credPath
              }

 If($Unreachable)
  {
   $SMTPServer = "smtp.office365.com" #Change To match your SMTP Server or Relay Connectors
   $SMTPPort   = "587" #Change to match your service port usually 587 or 25
   
   $Unreachable | Out-file "C:\UnreachableHosts.txt" -Force
   Start-Sleep -Seconds 2
   
   $Attachment = @()
   $Body = @()

   $From       = "emoreno@request-script.com" #Update with your data
   $To         = @("emoreno@request-script.com","DistributionList@request-script.com") #add emails or Distribution list here
   $Attachment += "C:\UnreachableHosts.txt" 
   $Subject    = "Unreachable Hosts"
   $Body       = "<b>Find Attached Report File</b><br>" #add any information that you consider relevant
   Send-MailMessage -From $From -to $To -Subject $Subject -Body $Body -BodyAsHtml -SmtpServer $SMTPServer `
   -port $SMTPPort -UseSsl -Credential $mailCreds -Attachments $Attachment #–DeliveryNotificationOption OnSuccess # Send Email
 }

  #Stop-Transcript

Regards

Erick Moreno

Nervill_mcbo For the error: You have to update the location of the server's file as it is looking for C:\servers.txt

The alternatives; Save a file containing all the IPs that did not respond or send an email notifying about it at the end. 

I can help you with that later, just confirm that after modifying the server's file location the script is able to complete using the task scheduler after that we can add functionality to the script as needed.