From 0afff9a9e2ef8561b6460a79025b59d64383c161 Mon Sep 17 00:00:00 2001
From: Vedant Bhamare <55763604+Dark-Knight11@users.noreply.github.com>
Date: Fri, 1 Oct 2021 18:16:36 +0530
Subject: [PATCH] Use new format for logging exceptions -
 services/modules/mqtt.py (#3183)

---
 supervisor/services/modules/mqtt.py | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/supervisor/services/modules/mqtt.py b/supervisor/services/modules/mqtt.py
index 0ad0b296d..d4177d0cb 100644
--- a/supervisor/services/modules/mqtt.py
+++ b/supervisor/services/modules/mqtt.py
@@ -68,10 +68,10 @@ class MQTTService(ServiceInterface):
     def set_service_data(self, addon: Addon, data: dict[str, Any]) -> None:
         """Write the data into service object."""
         if self.enabled:
-            _LOGGER.error(
-                "There is already a MQTT service in use from %s", self._data[ATTR_ADDON]
+            raise ServicesError(
+                f"There is already a MQTT service in use from {self._data[ATTR_ADDON]}",
+                _LOGGER.error,
             )
-            raise ServicesError()
 
         self._data.update(data)
         self._data[ATTR_ADDON] = addon.slug
@@ -82,8 +82,7 @@ class MQTTService(ServiceInterface):
     def del_service_data(self, addon: Addon) -> None:
         """Remove the data from service object."""
         if not self.enabled:
-            _LOGGER.warning("Can't remove not exists services")
-            raise ServicesError()
+            raise ServicesError("Can't remove not exists services", _LOGGER.warning)
 
         self._data.clear()
         self.save()