Don't ping link local broadcast (#188)

And don't fail the job immediately if the ping failed, the ARP entry may still exist anyway

Author: borrrden

environment/aws/topology_setup/test_server_platforms/ios_bridge.py

@@ -360,11 +360,20 @@ def __broadcast_ping_request(self) -> None:
                     continue
 
                 ip = addr[netifaces.AF_INET][0]["broadcast"]
+                if ip.startswith("169.254"):
+                    continue
+
                 click.echo(f"Broadcasting ping request on {interface} ({ip})")
-                subprocess.run(
-                    ["ping", ip, "-c", "3"], check=True, capture_output=True, text=True
+                result = subprocess.run(
+                    ["ping", ip, "-c", "3"], check=False, capture_output=True, text=True
                 )
 
+                if result.returncode != 0:
+                    click.secho(
+                        f"Failed to ping {ip} on {interface}: {result.stderr}",
+                        fg="yellow",
+                    )
+
     def get_ip(self, location: str) -> str:
         """
         Retrieve the IP address of the specified device.