fix(glances): use dedicated httpx client with longer timeout and surface API error messages

homeassistant/components/glances/__init__.py

@@ -26,8 +26,9 @@
     ConfigEntryNotReady,
     HomeAssistantError,
 )
-from homeassistant.helpers.httpx_client import get_async_client
+from homeassistant.helpers.httpx_client import create_async_httpx_client
 
+from .const import DEFAULT_TIMEOUT
 from .coordinator import GlancesConfigEntry, GlancesDataUpdateCoordinator
 
 PLATFORMS = [Platform.SENSOR]
@@ -65,7 +66,9 @@ async def async_unload_entry(hass: HomeAssistant, entry: GlancesConfigEntry) ->
 
 async def get_api(hass: HomeAssistant, entry_data: dict[str, Any]) -> Glances:
     """Return the api from glances_api."""
-    httpx_client = get_async_client(hass, verify_ssl=entry_data[CONF_VERIFY_SSL])
+    httpx_client = create_async_httpx_client(
+        hass, verify_ssl=entry_data[CONF_VERIFY_SSL], timeout=DEFAULT_TIMEOUT
+    )
     for version in (4, 3):
         api = Glances(
             host=entry_data[CONF_HOST],

homeassistant/components/glances/const.py

@@ -9,5 +9,6 @@
 DEFAULT_HOST = "localhost"
 DEFAULT_PORT = 61208
 DEFAULT_SCAN_INTERVAL = timedelta(seconds=60)
+DEFAULT_TIMEOUT = 30
 
 CPU_ICON = f"mdi:cpu-{64 if sys.maxsize > 2**32 else 32}-bit"

homeassistant/components/glances/coordinator.py

@@ -46,7 +46,7 @@ async def _async_update_data(self) -> dict[str, Any]:
         except exceptions.GlancesApiAuthorizationError as err:
             raise ConfigEntryAuthFailed from err
         except exceptions.GlancesApiError as err:
-            raise UpdateFailed from err
+            raise UpdateFailed(str(err)) from err
         # Update computed values
         uptime: datetime | None = None
         up_duration: timedelta | None = None

tests/components/glances/test_init.py

@@ -2,20 +2,22 @@
 
 from unittest.mock import MagicMock
 
+from freezegun.api import FrozenDateTimeFactory
 from glances_api.exceptions import (
     GlancesApiAuthorizationError,
     GlancesApiConnectionError,
     GlancesApiNoDataAvailable,
 )
 import pytest
 
-from homeassistant.components.glances.const import DOMAIN
+from homeassistant.components.glances.const import DEFAULT_SCAN_INTERVAL, DOMAIN
 from homeassistant.config_entries import ConfigEntryState
 from homeassistant.core import HomeAssistant
+from homeassistant.helpers.update_coordinator import UpdateFailed
 
 from . import MOCK_USER_INPUT
 
-from tests.common import MockConfigEntry
+from tests.common import MockConfigEntry, async_fire_time_changed
 
 
 async def test_successful_config_entry(hass: HomeAssistant) -> None:
@@ -53,6 +55,34 @@ async def test_setup_error(
     assert entry.state is entry_state
 
 
+async def test_update_error_includes_message(
+    hass: HomeAssistant,
+    freezer: FrozenDateTimeFactory,
+    mock_api: MagicMock,
+) -> None:
+    """Test that the underlying API error message is propagated to UpdateFailed."""
+    entry = MockConfigEntry(domain=DOMAIN, data=MOCK_USER_INPUT)
+    entry.add_to_hass(hass)
+
+    await hass.config_entries.async_setup(entry.entry_id)
+    await hass.async_block_till_done()
+    assert entry.state is ConfigEntryState.LOADED
+
+    mock_api.return_value.get_ha_sensor_data.side_effect = GlancesApiConnectionError(
+        "Connection to http://localhost:61209/api/4/all failed"
+    )
+    freezer.tick(DEFAULT_SCAN_INTERVAL)
+    async_fire_time_changed(hass)
+    await hass.async_block_till_done()
+
+    coordinator = entry.runtime_data
+    assert coordinator.last_update_success is False
+    assert isinstance(coordinator.last_exception, UpdateFailed)
+    assert "Connection to http://localhost:61209/api/4/all failed" in str(
+        coordinator.last_exception
+    )
+
+
 async def test_unload_entry(hass: HomeAssistant) -> None:
     """Test removing Glances."""
     entry = MockConfigEntry(domain=DOMAIN, data=MOCK_USER_INPUT)