Call Subscribe on systemd D-Bus connect to enable signals (#6659)

systemd only emits bus signals (including PropertiesChanged) when at
least one client has called Subscribe() on the Manager interface. On
regular HAOS systems, systemd-logind calls Subscribe which enables
signals for all bus clients. However, in environments without
systemd-logind (such as the Supervisor devcontainer with systemd), no
signals are emitted, causing the firewall unit wait to time out.

Explicitly calling Subscribe() has no downsides and makes it clear
that the Supervisor relies on these signals. There is no need to call
Unsubscribe() as systemd automatically tracks clients and stops
emitting signals when all subscribers have disconnected from the bus.

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

supervisor/dbus/systemd.py

@@ -112,6 +112,12 @@ class Systemd(DBusInterfaceProxy):
                 "No systemd support on the host. Host control has been disabled."
             )
 
+        if self.is_connected:
+            try:
+                await self.connected_dbus.Manager.call("subscribe")
+            except DBusError:
+                _LOGGER.warning("Could not subscribe to systemd signals")
+
     @property
     @dbus_property
     def startup_time(self) -> float:

tests/dbus/test_systemd.py

@@ -32,6 +32,18 @@ async def test_dbus_systemd_info(dbus_session_bus: MessageBus):
     assert systemd.startup_time == 45.304696
 
 
+async def test_subscribe_on_connect(
+    systemd_service: SystemdService, dbus_session_bus: MessageBus
+):
+    """Test that Subscribe is called on connect to enable signal emission."""
+    systemd_service.Subscribe.calls.clear()
+    systemd = Systemd()
+
+    await systemd.connect(dbus_session_bus)
+
+    assert systemd_service.Subscribe.calls == [()]
+
+
 async def test_reboot(systemd_service: SystemdService, dbus_session_bus: MessageBus):
     """Test reboot."""
     systemd_service.Reboot.calls.clear()

tests/dbus_service_mocks/systemd.py

@@ -656,6 +656,10 @@ class Systemd(DBusServiceMock):
     def PowerOff(self) -> None:
         """Power off host computer."""
 
+    @dbus_method()
+    def Subscribe(self) -> None:
+        """Subscribe to systemd signals."""
+
     @dbus_method()
     def StartUnit(self, name: "s", mode: "s") -> "o":
         """Start a service unit."""