Skip to content

Commit 1d0c55b

Browse files
waleedlatif1claude
andcommitted
fix: address the fifth review round
- The Ollama upgrade commands ran unprofiled, so Compose would have skipped the profile-scoped `ollama` service and left `OLLAMA_URL` pointing at nothing. Both commands now carry the install's `--profile`. - `start` / `stop` / `restart` are only whole-install operations on Compose. On a source checkout they manage just the Postgres and Redis containers, and on Kubernetes they print kubectl commands rather than acting. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015BwsJTEQRzWJaY4BRCkPZt
1 parent a9169b3 commit 1d0c55b

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

‎apps/docs/content/docs/platform/self-hosting/docker.mdx‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -169,7 +169,7 @@ docker compose -f "$COMPOSE_FILE" up -d --force-recreate simstudio
169169
| `sim-setup` | The setup wizard. `--quick` skips the prompts, `--dir <path>` picks the directory, `--mode compose\|dev\|k8s` picks the target (`dev` and `k8s` need a source checkout) |
170170
| `sim-setup status` | Show what is installed and healthy |
171171
| `sim-setup logs` | Follow logs. On a source checkout it prints where the logs are — the dev server streams in its own terminal — rather than following a stream |
172-
| `sim-setup start` / `stop` / `restart` | Bring the install up, down, or cycle it |
172+
| `sim-setup start` / `stop` / `restart` | Bring the install up, down, or cycle it. On a source checkout this covers only the managed Postgres and Redis containers — the dev server is yours to start and stop. On Kubernetes it prints the equivalent `kubectl` commands instead of acting |
173173
| `sim-setup update` | Pull or rebuild images and apply them. Compose installs only — it refuses on source checkouts and Kubernetes, which update through git and `helm upgrade` |
174174
| `sim-setup down` | Remove containers, keep the data |
175175
| `sim-setup reset` | Archive `.env` and wipe managed data |

‎apps/docs/content/docs/platform/self-hosting/upgrades.mdx‎

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -196,8 +196,10 @@ Inspect the result with `npx sim-setup logs`, which targets whichever Compose fi
196196
The CLI detects only those two files. An install started from `docker-compose.ollama.yml` is invisible to it — `update`, `logs`, and `status` all report no install — so upgrade that stack directly:
197197

198198
```bash
199-
docker compose -f docker-compose.ollama.yml pull
200-
docker compose -f docker-compose.ollama.yml up -d
199+
# Same --profile you installed with: gpu or cpu. The Ollama service is
200+
# profile-scoped, so an unprofiled command silently skips it.
201+
docker compose -f docker-compose.ollama.yml --profile gpu pull
202+
docker compose -f docker-compose.ollama.yml --profile gpu up -d
201203
```
202204

203205
<Callout type="warn">

0 commit comments

Comments
 (0)