search

ualex73 / monitor_docker

Monitor Docker containers from Home Assistant
Apache License 2.0
268 stars 34 forks source link

Error handling changes to the list of containers #25

Closed LordShadowen closed 3 years ago

LordShadowen commented 3 years ago

If the allinone contion is active and then there's a change to the list of containers in docker itself (adding one, removing one, etc...) the integration rebuilds all the sensors as separate entities instead of keeping them as one sensor per container.
Restarting HA reverts everything back to normal.

ualex73 commented 3 years ago

Ok, i didn't explicitly test that scenario. I will do that somewhere next week and then i will try to fix it.

ualex73 commented 3 years ago

Should be fixed in the latest commit, but not available via HACS.

ualex73 commented 3 years ago

@LordShadowen you got time to test it?

LordShadowen commented 3 years ago

Yes - I did. Yep, seems to work fine now

ualex73 commented 3 years ago

Cool, then we can close this one ;-)