voice-to-mermaid: remove vtm-ollama sidecar, default qwen3:8b#8
Conversation
…n3:8b Consolidates on Windows-native Ollama via host.docker.internal:11434: - Remove vtm-ollama sidecar container + ollama-data volume (was vestigial — api was already pointing at host.docker.internal by default) - Update default_model from gemma4:31b (did not exist) to qwen3:8b - Update model_filter to match models actually installed on Windows Ollama Part of the 2026-04-24 reproducible-infra consolidation — full context at simpleDiscordBot/.agents/plans/2026-04-24_reproducible-infra-consolidation/ Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
There was a problem hiding this comment.
Pull request overview
Removes the unused Ollama sidecar from the Voice-to-Mermaid backend Compose setup and updates the backend configuration to use an available default Ollama model (qwen3:8b) with a corrected model filter.
Changes:
- Deleted the
vtm-ollamaservice andollama-datavolume fromdocker-compose.yml, switching the API to use Windows-native Ollama viahost.docker.internal. - Added a dedicated
local-aiDocker network for the API container. - Updated
config.yamlto setdefault_modeltoqwen3:8band refreshedmodel_filterto models intended to exist.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| voice-to-mermaid/backend/docker-compose.yml | Removes Ollama sidecar, points API at host Ollama, adds local network, adds architecture guidance comments. |
| voice-to-mermaid/backend/config.yaml | Changes default Ollama model and updates the UI model filter list with additional guidance comments. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| capabilities: [gpu] | ||
| # Voice-to-Mermaid — LOCAL MACHINE ONLY (requires GPU) | ||
| # ⚠️ GPU GUARD: This compose file requires NVIDIA GPU hardware. | ||
| # ⚠️ DO NOT deploy to the VPS (187.77.216.108) — it has no GPU. |
There was a problem hiding this comment.
The comment hardcodes a public VPS IP address. Even in comments, committing infrastructure IPs can increase attack surface and becomes stale quickly; consider referencing a hostname/alias (e.g. "the VPS") or pointing to an internal inventory/runbook instead of embedding the raw IP.
| # ⚠️ DO NOT deploy to the VPS (187.77.216.108) — it has no GPU. | |
| # ⚠️ DO NOT deploy to the VPS — it has no GPU. |
| # Voice-to-Mermaid — LOCAL MACHINE ONLY (requires GPU) | ||
| # ⚠️ GPU GUARD: This compose file requires NVIDIA GPU hardware. | ||
| # ⚠️ DO NOT deploy to the VPS (187.77.216.108) — it has no GPU. | ||
| # ⚠️ See infrastructure/local/README.md for local machine service registry. |
There was a problem hiding this comment.
This comment references infrastructure/local/README.md, but that path does not exist in the repository. Please update the pointer to an existing doc path or remove the reference to avoid broken guidance.
| # ⚠️ See infrastructure/local/README.md for local machine service registry. | |
| # ⚠️ Local machine only: keep service guidance in repository docs up to date. |
| # See .agents/runbooks/troubleshoot-ollama/SKILL.md for the single-instance | ||
| # architecture rules. |
There was a problem hiding this comment.
This comment points to .agents/runbooks/troubleshoot-ollama/SKILL.md, but there is no .agents directory in the repo. Please update to an existing runbook location or remove the reference so readers can actually find the architecture rules.
| # See .agents/runbooks/troubleshoot-ollama/SKILL.md for the single-instance | |
| # architecture rules. | |
| # Keep this as a single-instance setup: use the existing Windows-native | |
| # Ollama service and do not reintroduce a Compose-managed Ollama sidecar here. |
| # Models shown in the UI picker. Must exist in Windows Ollama | ||
| # (C:\Users\jaewi\.ollama is not used — see OLLAMA_MODELS=D:\ollama\models). |
There was a problem hiding this comment.
The comment uses a user-specific Windows path (C:\Users\jaewi...) and a specific drive letter for OLLAMA_MODELS. Consider making this guidance portable (e.g., %USERPROFILE%\.ollama, or describing how to locate/set OLLAMA_MODELS) so it applies to other developers/machines.
| # Models shown in the UI picker. Must exist in Windows Ollama | |
| # (C:\Users\jaewi\.ollama is not used — see OLLAMA_MODELS=D:\ollama\models). | |
| # Models shown in the UI picker. Must exist in Ollama's configured models | |
| # directory (by default on Windows, under %USERPROFILE%\.ollama; if | |
| # OLLAMA_MODELS is set, use that location instead). |
Summary
vtm-ollamasidecar container +ollama-datavolume — api already defaulted tohost.docker.internal:11434(Windows-native Ollama)config.yaml:default_modelwasgemma4:31b(did not exist), nowqwen3:8b.model_filterupdated for models that actually exist.Context
2026-04-24 reproducibility consolidation. Before this change the backend had:
Full context: simpleDiscordBot/.agents/plans/2026-04-24_reproducible-infra-consolidation/plan.md
Test plan
🤖 Generated with Claude Code