search

ualex73 / monitor_docker

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

Can only restart container if container list explicitly specified #58

Closed derekoharrow closed 3 years ago

derekoharrow commented 3 years ago

I can use the restart service no problem if the container is specifically listed in the containers section in configuration.yaml.

However, the default for monitor_docker is to monitor all containers if you don't specify a list.

If you use the default, to monitor all containers, and don't specify a list, then the restart service doesn't work. It complains, for example:

[custom_components.monitor_docker.switch] Service restart failed, container 'zigbee2mqtt' is not configured

The desired behaviour would be to enable you to restart a service even if you used the default option and didn't list each docker container individually in the configuration.

ualex73 commented 3 years ago

Thanks for the bug report, it should be fixed in the GitHub version. Can you test it out? If it works fine, I will push it via HACS.

ualex73 commented 3 years ago

Version 1.9 has been released, so we can close this one.