From 9e5a9e014169f8a311085334b118227862fb63fc Mon Sep 17 00:00:00 2001 From: Sven Sager Date: Fri, 13 Mar 2026 09:07:46 +0100 Subject: [PATCH] chore: Add logging for D-Bus publishing and connection closure Added informational log messages to track when `REVPI_DBUS_NAME` is published and its connection is closed in both `BusProviderIos1` and `BusProviderMiddleware1`. Signed-off-by: Sven Sager --- src/revpi_middleware/dbus_ios1/bus_provider_ios1.py | 2 ++ .../dbus_middleware1/bus_provider_middleware1.py | 2 ++ 2 files changed, 4 insertions(+) diff --git a/src/revpi_middleware/dbus_ios1/bus_provider_ios1.py b/src/revpi_middleware/dbus_ios1/bus_provider_ios1.py index d7284d8..707fba4 100644 --- a/src/revpi_middleware/dbus_ios1/bus_provider_ios1.py +++ b/src/revpi_middleware/dbus_ios1/bus_provider_ios1.py @@ -92,6 +92,7 @@ class BusProviderIos1(Thread): *lst_interfaces, ) self.published.set() + log.info(f"published {REVPI_DBUS_NAME} on {self._bus_address}") except Exception as e: log.error(f"can not publish dbus {REVPI_DBUS_NAME}: {e}") @@ -109,6 +110,7 @@ class BusProviderIos1(Thread): bus.con.close() self._modio.cleanup() + log.info(f"closed {REVPI_DBUS_NAME} connection to {self._bus_address}") log.debug("leave BusProviderIos1.run") diff --git a/src/revpi_middleware/dbus_middleware1/bus_provider_middleware1.py b/src/revpi_middleware/dbus_middleware1/bus_provider_middleware1.py index 040f21a..fb91249 100644 --- a/src/revpi_middleware/dbus_middleware1/bus_provider_middleware1.py +++ b/src/revpi_middleware/dbus_middleware1/bus_provider_middleware1.py @@ -55,6 +55,7 @@ class BusProviderMiddleware1(Thread): *lst_interfaces, ) self.published.set() + log.info(f"published {REVPI_DBUS_NAME} on {self._bus_address}") except Exception as e: log.error(f"can not publish dbus {REVPI_DBUS_NAME}: {e}") @@ -64,6 +65,7 @@ class BusProviderMiddleware1(Thread): log.error(f"can not run dbus mainloop: {e}") bus.con.close() + log.info(f"closed {REVPI_DBUS_NAME} connection to {self._bus_address}") # Clean up all interfaces for interface in lst_interfaces: