ualex73 / monitor_docker

Monitor Docker containers from Home Assistant

Apache License 2.0

268 stars 34 forks source link

Uptime sensor issue // Feature Request #2

Closed apedance closed 4 years ago

apedance commented 4 years ago

Hey,

again.

The uptime sensor seems to have issues.

How can I help and provide logs to you?

///////////////

Feature Request

Can you pass on a basic naming scheme for the sensors?

ovpn_fam CPU Usage containername Memory Used containername Memory % ...

Glances does this well.

Cheers o/

ualex73 commented 4 years ago

For the uptime I pushed a fix to github. For the sensor name best to remove ‘sensorname’ from your configuration. I made in the fix also the name of a sensor ‘ ’. So now it a bit more generic and does not follow my personal setup 🤪 Thanks for reporting the bug and if you find more, please let me know. Same applies for feature requests, then I will check what I can do.

apedance commented 4 years ago

fixed

Great work.

Will mess around with it further.

Lovely!

Cheers o/

ualex73 commented 4 years ago

Thanks for the confirmation it works good.

BTW: the total memory used by docker containers in glances and monitor_docker will differ, because glances seem to count also cache memory (this is released when needed) and i only count real memory (on my system Glances show 17GB and my memory shows 7GB ;-)).