Commit b25f978 prohibiting reboot if WiFi is not connected? about ginlong-solis HOT 3 CLOSED

I recommend to remove all timeouts etc. If no API client is connected for 15 minutes, ESPhome will reboot the device anyway.

I'm currently thinking about resetting the WDT only if there is active ModBus traffic and API clients connected, something like this:

--- a/solis-esphome-emw3080.yaml
+++ b/solis-esphome-emw3080.yaml
@@ -87,15 +87,6 @@
               switch.is_on: led_green_net
             then:
               - switch.turn_off: led_green_net
-  - interval: 280s
-    then:
-      if:
-        condition:
-          wifi.connected:
-        then:
-          - switch.turn_on: wdt_reset
-          - delay: 100ms
-          - switch.turn_off: wdt_reset
 
 script:
   - mode: restart
@@ -104,6 +95,13 @@
       - switch.turn_on: led_orange_com
       - delay: 500ms
       - switch.turn_off: led_orange_com
+      - if:
+          condition:
+            api.connected:
+          then:
+            - switch.turn_on: wdt_reset
+            - delay: 100ms
+            - switch.turn_off: wdt_reset
 
 uart:
   id: modbus_uart

Please do not expect any significant updates from me during the next 2-3 weeks.

from ginlong-solis.

Sounds like a good plan, any way to also detect CRC Check failed! and in return reboot when that happens? (somewhat off topic and should probably be asked here #4)

Please do not expect any significant updates from me during the next 2-3 weeks.

Ok no worries, I have a quite OK working setup atm with the solution I came up with here #4 (comment)

from ginlong-solis.

Just a small update regarding this ticket, with my HA automation that reboots the stick if the CRC Check failed!, that has now occurred 18 times in the past 5 days. Out of these 18 restarts the stick has returned instantly 16 times and only 2 times has it been offline for a longer period, however I guess that is due to the already known WiFi instability that has already been reported and discusses in other tickets.

So, with that information on hand I assume the b25f978 commit does not prohibit rebooting as I first suspected.

If you wanna close this ticket since I seem to have proved myself wrong that is fine, but I guess you might wanna keep it open due to the changes you have planned here #17 (comment)

from ginlong-solis.