From dc35c9c3d0faaff95f7b1739a1cc003508677179 Mon Sep 17 00:00:00 2001 From: Alexander Date: Mon, 5 Aug 2024 06:46:08 +0200 Subject: [PATCH 1/4] Update README.md --- README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index 9482485..935fd26 100644 --- a/README.md +++ b/README.md @@ -2,8 +2,8 @@ [![hacs_badge](https://img.shields.io/badge/HACS-Default-orange.svg?style=for-the-badge)](https://github.com/hacs/integration) -NOTE: For Home Assistant 2024.8 and up you need to modify the URL from "tcp://" to "http://" to make it work - +NOTE: For Home Assistant 2024.8, which uses aiohttp 3.10.1, they broke most of the monitor_docker functionality. This is reported in the aiohttp https://github.com/aio-libs/aiohttp/issues/8600 ticket. The problem is in the underlying aiohttp, so we cannot fix it. To workaround this, use the URL configuration item and point to a "http://ip:port" of your Docker instance. If you used "tcp://" before, replace it with "http://". +i ## About This repository contains the Monitor Docker component I developed for monitoring my Docker environment from [Home-Assistant](https://www.home-assistant.io). It is inspired by the Sander Huisman [Docker Monitor](https://github.com/Sanderhuisman/docker_monitor), where I switched mainly from threads to asyncio and put my own wishes/functionality in. Feel free to use the component and report bugs if you find them. If you want to contribute, please report a bug or pull request and I will reply as soon as possible. From 3223b797c5f395e0aeb8df8f12572e7ff561da8a Mon Sep 17 00:00:00 2001 From: Alexander Date: Mon, 5 Aug 2024 06:51:08 +0200 Subject: [PATCH 2/4] Update README.md --- README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index 935fd26..d556792 100644 --- a/README.md +++ b/README.md @@ -2,8 +2,8 @@ [![hacs_badge](https://img.shields.io/badge/HACS-Default-orange.svg?style=for-the-badge)](https://github.com/hacs/integration) -NOTE: For Home Assistant 2024.8, which uses aiohttp 3.10.1, they broke most of the monitor_docker functionality. This is reported in the aiohttp https://github.com/aio-libs/aiohttp/issues/8600 ticket. The problem is in the underlying aiohttp, so we cannot fix it. To workaround this, use the URL configuration item and point to a "http://ip:port" of your Docker instance. If you used "tcp://" before, replace it with "http://". -i +**NOTE: Home Assistant 2024.8 broke most of the monitor_docker functionality by using the latest aiohttp 3.10.1. This is reported in https://github.com/aio-libs/aiohttp/issues/8600 ticket. The problem is in the underlying aiohttp, so we cannot fix it. To workaround this (or stay on 2024.7), use the URL configuration item and point to a _http://ip:port_ of your Docker instance. If you used _tcp://_ before, replace it with _http://_.** + ## About This repository contains the Monitor Docker component I developed for monitoring my Docker environment from [Home-Assistant](https://www.home-assistant.io). It is inspired by the Sander Huisman [Docker Monitor](https://github.com/Sanderhuisman/docker_monitor), where I switched mainly from threads to asyncio and put my own wishes/functionality in. Feel free to use the component and report bugs if you find them. If you want to contribute, please report a bug or pull request and I will reply as soon as possible. From 55b8f1810884f82f96271a432730efe386613d1a Mon Sep 17 00:00:00 2001 From: Alexander Date: Thu, 8 Aug 2024 10:18:23 +0200 Subject: [PATCH 3/4] Update README.md --- README.md | 26 +++++++++++++++++++++++++- 1 file changed, 25 insertions(+), 1 deletion(-) diff --git a/README.md b/README.md index d556792..1811a75 100644 --- a/README.md +++ b/README.md @@ -2,7 +2,31 @@ [![hacs_badge](https://img.shields.io/badge/HACS-Default-orange.svg?style=for-the-badge)](https://github.com/hacs/integration) -**NOTE: Home Assistant 2024.8 broke most of the monitor_docker functionality by using the latest aiohttp 3.10.1. This is reported in https://github.com/aio-libs/aiohttp/issues/8600 ticket. The problem is in the underlying aiohttp, so we cannot fix it. To workaround this (or stay on 2024.7), use the URL configuration item and point to a _http://ip:port_ of your Docker instance. If you used _tcp://_ before, replace it with _http://_.** +## Home Assistant 2024.8 Problem + +Home Assistant 2024.8 broke most of the monitor_docker functionality by using the latest aiohttp 3.10.1. This is reported in https://github.com/aio-libs/aiohttp/issues/8600 ticket. The problem is in the underlying aiohttp, so we cannot fix it. Best to stay for the moment on 2024.7, or apply the workaround as shown below. + +1) **When no URL is specified, using the local unix socket (default)** + +Solution is to modify docker daemon to listen on a localhost port (my Home Assistant container uses the `host` network), restart docker daemon, then configure the `url` key in monitor_docker config. + +https://docs.docker.com/config/daemon/remote-access/ + +``` +$ cat /etc/docker/daemon.json +{ + "hosts": ["unix:///var/run/docker.sock", "tcp://127.0.0.1:2375"] +} +``` + +configuration.yaml: +``` +monitor_docker: + url: "http://localhost:2375" +``` + +2) **When using the URL configuration** +Change the URL to _http://ip:port_ of your Docker instance. If you used _tcp://_ before, replace it with _http://_. ## About From a4cb6a8b55ecc75f68348c592b51f1eb2e858dba Mon Sep 17 00:00:00 2001 From: Alexander Date: Sat, 10 Aug 2024 21:13:36 +0200 Subject: [PATCH 4/4] Update README.md --- README.md | 26 ++------------------------ 1 file changed, 2 insertions(+), 24 deletions(-) diff --git a/README.md b/README.md index 1811a75..cc6654d 100644 --- a/README.md +++ b/README.md @@ -2,31 +2,9 @@ [![hacs_badge](https://img.shields.io/badge/HACS-Default-orange.svg?style=for-the-badge)](https://github.com/hacs/integration) -## Home Assistant 2024.8 Problem +## Home Assistant 2024.8.0 -Home Assistant 2024.8 broke most of the monitor_docker functionality by using the latest aiohttp 3.10.1. This is reported in https://github.com/aio-libs/aiohttp/issues/8600 ticket. The problem is in the underlying aiohttp, so we cannot fix it. Best to stay for the moment on 2024.7, or apply the workaround as shown below. - -1) **When no URL is specified, using the local unix socket (default)** - -Solution is to modify docker daemon to listen on a localhost port (my Home Assistant container uses the `host` network), restart docker daemon, then configure the `url` key in monitor_docker config. - -https://docs.docker.com/config/daemon/remote-access/ - -``` -$ cat /etc/docker/daemon.json -{ - "hosts": ["unix:///var/run/docker.sock", "tcp://127.0.0.1:2375"] -} -``` - -configuration.yaml: -``` -monitor_docker: - url: "http://localhost:2375" -``` - -2) **When using the URL configuration** -Change the URL to _http://ip:port_ of your Docker instance. If you used _tcp://_ before, replace it with _http://_. +Home Assistant 2024.8.0 is not compatible with monitor_docker, please upgrade to Home Assistant 2024.8.1 ## About