-
-
Notifications
You must be signed in to change notification settings - Fork 33.4k
Ping integration warns when a manual polling method has been used #140319
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
Hey there @jpbede, mind taking a look at this issue as it has been labeled with an integration ( Code owner commandsCode owners of
(message by CodeOwnersMention) ping documentation |
How is your Ping config entry configured, how many ICMP requests do you send?
|
Ah I see - I've got it configured to be 15 pings (and consider home to be 320 seconds). I suppose there's no way to complete in under 10 seconds or change the 10 second config as it's hardcoded: core/homeassistant/helpers/entity.py Lines 1312 to 1314 in 2d09679
I suppose one option is to drop the amount of pings to bring it to completion in under 10s, but I have some devices which can be a little laggy to reply. What do you think? |
I suppose one approach would be to fire off a timer/task and update the sensor on completion of it, rather than when asked to by HA, but I'm not familiar enough with HA's internals to know if that's a sensible approach |
Yea something like that. We need to postpone the first ping after the start. |
Does #143616 look along the right lines? |
The problem
When I configure the ping sensor to manually update (by defining a custom polling interval), I receive these warnings in the HA log:
Should the ping integration change to mark its sensor as updated, even when "Enable Polling for Changes" is disabled?
What version of Home Assistant Core has the issue?
core-2025.3.0
What was the last working version of Home Assistant Core?
No response
What type of installation are you running?
Home Assistant Core
Integration causing the issue
ping
Link to integration documentation on our website
https://www.home-assistant.io/integrations/ping
Diagnostics information
No response
Example YAML snippet
Anything in the logs that might be useful for us?
Additional information
No response
The text was updated successfully, but these errors were encountered: