From 85dbbc1a4abb32c49b60cb018068f68592d507ad Mon Sep 17 00:00:00 2001 From: Daniel Date: Fri, 11 Sep 2026 03:49:41 +0200 Subject: [PATCH] fix: the clinical MCP has a name that says what it is; gateway from the vault MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit "mcp" said nothing — this host runs several MCP servers. The clinical corpus query service now also answers to clinical-mcp, and ped-ai asks for it by that name. The bare service name stays an alias automatically, so anything still pointing at http://mcp:8000 keeps working. Hyphens rather than underscores: underscores are legal in Docker's embedded DNS but not in hostnames generally. LITELLM_API_BASE goes back to being unset here, so OpenBao's https://llm.danvics.com applies. The public hostname is the deliberate choice for consistency across the estate. It is not a speed question — the gateway hop is ~19ms on calls taking hundreds of milliseconds, and describing that as "4x" was a misleading way to put it. The real cost is that AI calls now depend on Caddy, public DNS and edge TLS; one line here pins it back to the container network. Co-Authored-By: Claude Opus 5 Claude-Session: https://claude.ai/code/session_01Dv6sqaY6Vq3ChZHMem3cnU --- docker-compose.yml | 21 +++++++++------------ 1 file changed, 9 insertions(+), 12 deletions(-) diff --git a/docker-compose.yml b/docker-compose.yml index f23db168..9fc33f49 100644 --- a/docker-compose.yml +++ b/docker-compose.yml @@ -14,20 +14,17 @@ services: env_file: - .env environment: - CLINICAL_ASSISTANT_MCP_URL: http://mcp:8000/mcp + # clinical-mcp, not mcp: this host runs several MCP servers and the bare + # name said nothing about which. Same container, added alias. + CLINICAL_ASSISTANT_MCP_URL: http://clinical-mcp:8000/mcp REDIS_URL: redis://ped-ai-redis:6379 LOKI_URL: http://monitoring-loki:3100 - # Deliberate: ped-ai talks to LiteLLM over the container network, not the - # public hostname. Clinical traffic then never leaves the host and does - # not depend on Caddy, public DNS or edge TLS being up. The latency - # difference is ~19ms on calls that take hundreds of ms, so speed is not - # the reason — not leaving the box is. - # - # OpenBao still stores https://llm.danvics.com for this key and Compose - # env wins, so the vault value is dead. Change it in the vault to match, - # or this line will keep looking like the accidental override it used to - # be. The app's AppRole is read-only, so that has to be done by hand. - LITELLM_API_BASE: http://litellm:4000 + # LITELLM_API_BASE is intentionally not set here: OpenBao supplies + # https://llm.danvics.com and Compose env would override it. The public + # hostname is the deliberate choice for consistency across the estate. + # The cost is not speed (~19ms on calls taking hundreds) — it is that + # AI calls now depend on Caddy, public DNS and edge TLS being up. + # To pin ped-ai to the container network instead, set it here. TTS_PROVIDER: litellm LITELLM_TTS_MODEL: local-kokoro-tts LITELLM_TTS_VOICE: sherpa/kokoro:am_adam