core: services: ping: Keep port watcher running after unexpected errors

[joaoantoniocardoso]

Summary

• Catch unexpected errors in the Ping PortWatcher loop so discovery keeps running.
• Add a regression test that a transient add_ping360 failure does not stop watching.

Follow-up to #4036: that PR fixes the list_ips() KeyError, but any other uncaught exception in the watch loop still permanently kills sensor discovery because sensor_manager() is started with a bare asyncio.create_task.

Test plan

• pytest core/services/ping/test_portwatcher.py
• Confirm Ping service continues discovering after SoftAP/uap0 churn on a vehicle

---

Supersedes #4040 (recreated from fork joaoantoniocardoso/BlueOS-docker instead of same-repo head).

[github-actions]

Automated PR Review

0. Summary

• Verdict: LGTM

Wraps the body of PortWatcher.start_watching's while True loop in try/except Exception + logger.exception, so any unexpected error (e.g. transient add_ping360/list_ips failures) no longer permanently kills the sensor-discovery task. Adds a focused regression test (core/services/ping/test_portwatcher.py) that confirms a raised KeyError in add_ping360 does not stop the watch loop, using asyncio.CancelledError (a BaseException on 3.11, so unaffected by except Exception) to break out on the second iteration.

No further comments, nice job 👍

Generated by PR Review Bot. This is advisory, a human reviewer must still approve.