From 11fcaffa97823f1407ae24d0a17ef80609d7ca90 Mon Sep 17 00:00:00 2001
From: ukkopahis <95980324+ukkopahis@users.noreply.github.com>
Date: Fri, 29 Apr 2022 10:18:11 +0300
Subject: [PATCH] telegraf: remove unused mosquitto dependency

Seems like remnant misconfiguration from the beginning.

Fixes docker-compose.yml to be valid even if you don't choose mosquitto
fron the menu.

Added documentation about adding the mosquitto 'depends_on' back if you
use the mqtt_consumer addition.

Resolves #550
---
 .templates/telegraf/service.yml | 1 -
 docs/Containers/Telegraf.md     | 4 ++++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/.templates/telegraf/service.yml b/.templates/telegraf/service.yml
index 1b5b97e95..2e0eb2b90 100644
--- a/.templates/telegraf/service.yml
+++ b/.templates/telegraf/service.yml
@@ -13,5 +13,4 @@ telegraf:
     - /var/run/docker.sock:/var/run/docker.sock:ro
   depends_on:
     - influxdb
-    - mosquitto
 
diff --git a/docs/Containers/Telegraf.md b/docs/Containers/Telegraf.md
index ca3a8ed17..77a8a09f8 100644
--- a/docs/Containers/Telegraf.md
+++ b/docs/Containers/Telegraf.md
@@ -249,6 +249,10 @@ $ docker-compose restart telegraf
 
 The `grep` strips comment lines and the `sudo tee` is a safe way of appending the result to `telegraf.conf`. The `restart` causes Telegraf to notice the change.
 
+This result in the Telegraf service depending on the Modquitto service, as such
+you may want to edit `docker-compose.yml` and add `- mosquitto` to the
+`depends_on:` of the mosquitto service.
+
 ## <a name="cleanSlate"></a>Getting a clean slate
 
 ### <a name="resetDB"></a>Erasing the persistent storage area