izackp/webterm
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix sparkline slug-to-service mapping initialization

Browse Source 
- Initialize _slug_to_service as instance variable in __init__
- Add logging to help debug empty sparklines
- Log the mapping at startup
This commit is contained in:
GitHub Copilot
2026-01-24 12:13:38 +00:00
parent 231bf69b3d
commit 92651e236b
1 changed files with 13 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/textual_webterm/local_server.py
+13 -3
View File
@@ -228,6 +228,7 @@ class LocalServer:
# Docker stats collector (only used in compose mode) # Docker stats collector (only used in compose mode)
self._docker_stats: DockerStatsCollector | None = None self._docker_stats: DockerStatsCollector | None = None
self._slug_to_service: dict[str, str] = {}
@property @property
def app_count(self) -> int: def app_count(self) -> int:
@@ -348,13 +349,13 @@ class LocalServer:
self._docker_stats = DockerStatsCollector() self._docker_stats = DockerStatsCollector()
if self._docker_stats.available: if self._docker_stats.available:
# Pass service names (not slugs) for Docker matching # Pass service names (not slugs) for Docker matching
# Also pass slug->name mapping so we can look up by slug
service_names = [app.name for app in self._landing_apps] service_names = [app.name for app in self._landing_apps]
self._docker_stats.start(service_names) self._docker_stats.start(service_names)
# Create slug->name mapping for lookups # Create slug->name mapping for lookups
self._slug_to_service: dict[str, str] = { self._slug_to_service = {
app.slug: app.name for app in self._landing_apps app.slug: app.name for app in self._landing_apps
} }
log.info("Slug to service mapping: %s", self._slug_to_service)
stack.push_async_callback(self._docker_stats.stop) stack.push_async_callback(self._docker_stats.stop)
site = web.TCPSite(runner, self.host, self.port) site = web.TCPSite(runner, self.host, self.port)
@@ -672,8 +673,17 @@ class LocalServer:
values: list[float] = [] values: list[float] = []
if self._docker_stats: if self._docker_stats:
# Container param is slug, but stats are stored by service name # Container param is slug, but stats are stored by service name
service_name = getattr(self, "_slug_to_service", {}).get(container, container) service_name = self._slug_to_service.get(container, container)
values = self._docker_stats.get_cpu_history(service_name) values = self._docker_stats.get_cpu_history(service_name)
if not values:
log.debug(
"No CPU history for container=%s service=%s (available=%s)",
container,
service_name,
list(self._docker_stats._cpu_history.keys()),
)
else:
log.debug("Docker stats collector not available")
svg = render_sparkline_svg(values, width=width, height=height) svg = render_sparkline_svg(values, width=width, height=height)
headers = {"Cache-Control": "no-cache, max-age=0"} headers = {"Cache-Control": "no-cache, max-age=0"}