Skip to content
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

No need for logs when devices are off #206

Open
andreasc1 opened this issue Jan 11, 2023 · 5 comments
Open

No need for logs when devices are off #206

andreasc1 opened this issue Jan 11, 2023 · 5 comments
Labels
bug Something isn't working

Comments

@andreasc1
Copy link

Describe the bug (描述一下问题)
Hi, log files of HA are flooded with communication error messages. no need to flood the log files with all these kind of messages when the devices are off:

Screenshots / Logs / Pcap File(屏幕截图/日志/抓包文件)
If applicable, add screenshots or your home-assistant log file or pcap file to help explain your problem.
请提交相关截图,日志,抓包让开发者能够更快的解决问题
2023-01-11 13:54:35.736 ERROR (SyncWorker_11) [msmart.lan] Connect Error: 192.168.1.92:6444 timed out
2023-01-11 13:54:35.738 ERROR (SyncWorker_10) [msmart.lan] Connect Error: 192.168.1.93:6444 timed out
2023-01-11 13:54:35.738 ERROR (SyncWorker_5) [msmart.lan] Connect Error: 192.168.1.70:6444 timed out
2023-01-11 13:54:35.739 ERROR (SyncWorker_12) [msmart.lan] Connect Error: 192.168.1.91:6444 timed out
2023-01-11 13:54:35.743 ERROR (SyncWorker_11) [msmart.lan] Sokcet is None: 192.168.1.92:6444
2023-01-11 13:54:35.746 ERROR (SyncWorker_10) [msmart.lan] Sokcet is None: 192.168.1.93:6444
2023-01-11 13:54:35.748 ERROR (SyncWorker_5) [msmart.lan] Sokcet is None: 192.168.1.70:6444
2023-01-11 13:54:35.751 ERROR (SyncWorker_12) [msmart.lan] Sokcet is None: 192.168.1.91:6444
2023-01-11 13:54:35.753 ERROR (SyncWorker_11) [msmart.security] unexpected data length
2023-01-11 13:54:35.757 ERROR (SyncWorker_5) [msmart.security] unexpected data length
2023-01-11 13:54:35.760 ERROR (SyncWorker_12) [msmart.security] unexpected data length
2023-01-11 13:54:35.761 ERROR (SyncWorker_11) [msmart.lan] Authentication failed for None -> 192.168.1.92:6444 retries: 0 time: 238416.93
2023-01-11 13:54:35.767 ERROR (SyncWorker_5) [msmart.lan] Authentication failed for None -> 192.168.1.70:6444 retries: 0 time: 238416.94
2023-01-11 13:54:35.769 ERROR (SyncWorker_12) [msmart.lan] Authentication failed for None -> 192.168.1.91:6444 retries: 0 time: 238416.9
Versions (版本信息)

  • Home Assistant version: 2023.1
  • Midea msmart version: latest
@andreasc1 andreasc1 added the bug Something isn't working label Jan 11, 2023
@mill1000
Copy link

How is the integration supposed to know the device is "off", instead of experiencing a temporary loss of connectivity?

@liakjim
Copy link

liakjim commented Apr 5, 2023

+1

@andreasc1
Copy link
Author

How is the integration supposed to know the device is "off", instead of experiencing a temporary loss of connectivity?

yes i get what you are saying. If the latest status of the device is off (or idle, i don't know how it is defined) then it should not export any connectivity errors? but i understand that if there's actually an error the integratio wont report it..

@liakjim
Copy link

liakjim commented Apr 5, 2023

There are also the following warnings:

Logger: msmart.device.AC.appliance
Source: /usr/local/lib/python3.10/site-packages/msmart/device/AC/appliance.py:114
First occurred: 9:37:53 AM (19 occurrences)
Last logged: 9:49:54 AM

Got Null from 192.168.3.8:6444 Version: 2 Count: 0 Spend time: 3.01
Got Null from 192.168.3.8:6444 Version: 2 Count: 0 Spend time: 2.01
Got Null from 192.168.3.8:6444 Version: 2 Count: 0 Spend time: 5.01
Got Null from 192.168.3.8:6444 Version: 2 Count: 0 Spend time: 3.0
Got Null from 192.168.3.8:6444 Version: 2 Count: 0 Spend time: 5.0

and the following error:

Logger: msmart.lan
Source: /usr/local/lib/python3.10/site-packages/msmart/lan.py:70
First occurred: 9:38:18 AM (1 occurrences)
Last logged: 9:38:18 AM

Send 192.168.3.126:39932 -> 192.168.3.8:6444 retries: 0 time: 165.03 Error: [Errno 32] Broken pipe

@mill1000
Copy link

yes i get what you are saying. If the latest status of the device is off (or idle, i don't know how it is defined) then it should not export any connectivity errors? but i understand that if there's actually an error the integratio wont report it..

At least with my unit, when it's "off" the network is still connected and reachable by the integration (temperatures are still polled). The only time the device is unreachable is when power is completely removed (i.e. unplugged) at which point I just disable the integration in HA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants