-
Notifications
You must be signed in to change notification settings - Fork 114
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
Make monitoring of apt optional #37
Comments
For a system running Hass itself you have all the sensors with a build in sensor. This script just reproduce the component. Does it give any errors for the apt updates or does it show just 0 updates? |
The issue in docker is that it makes no sense to try to detect it's apt status, as it's apt cache is always clean. That's why I didn't want to install python3-apt in the first place. Now I tried and it would install lot of dependencies, as that base python image is stripped down as much as possible. I am not sure, whether it would be possible to talk to apt of host system from docker container to get it's apt status. BTW: Could you direct me where to find those built in sensors? |
For a device running Hass you could use system-monitor https://www.home-assistant.io/integrations/systemmonitor |
@Sennevds Thanks a lot! It works. |
I am afraid, I was too fast reporting it works correctly. The missing quote got lost here: 94e7138#diff-1f786a78bd19509da3007e84eaf2df1fL85 |
I've changed the code but can't test right now could you test it? |
Yep, the missing quote is back and HA can parse it. |
While I agree, that monitoring of apt updates can be useful on many systems, on pre-built systems (like Hassio itself), it is not very useful and when you want to run system-sensors in docker, it doesn't make sense at all.
I want to monitor my RPi 3 running Hassio, but I am reluctant to install something directly into it's OS.
So I tried to run system-sensors as a docker container and apart from those apt updates, it works just fine without much effort.
I can provide a short how-to for anyone interested, but I was lazy and didn't even bother to create new docker image for it, so it is a bit dirty ;-)
The text was updated successfully, but these errors were encountered: