Skip to content

Commit 1daf58e

Browse files
waleedlatif1claude
andcommitted
fix: address the sixth review round
- The Ollama upgrade commands hard-coded `--profile gpu` while the comment said to use the install's profile. It is a variable now. - `sim-setup reset` on Kubernetes runs `helm uninstall`, which leaves the Postgres volumes behind — they come from a StatefulSet's `volumeClaimTemplates`, which Kubernetes does not delete. The row promised a full data wipe. - Noted that the command table drops the `npx` prefix for brevity. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_015BwsJTEQRzWJaY4BRCkPZt
1 parent 1d0c55b commit 1daf58e

2 files changed

Lines changed: 7 additions & 6 deletions

File tree

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,7 @@ docker compose -f "$COMPOSE_FILE" up -d --force-recreate simstudio
162162

163163
## The `sim-setup` CLI
164164

165-
`npx sim-setup` also manages an existing install created from the production or local-development Compose files. It does not detect or manage an Ollama-stack install. Run it from the directory holding your `.env` and Compose file.
165+
`npx sim-setup` also manages an existing install created from the production or local-development Compose files. It does not detect or manage an Ollama-stack install. Run it from the directory holding your `.env` and Compose file. The table drops the `npx` prefix for brevity — keep it unless you installed the package globally.
166166

167167
| Command | What it does |
168168
|---|---|
@@ -172,7 +172,7 @@ docker compose -f "$COMPOSE_FILE" up -d --force-recreate simstudio
172172
| `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 |
175-
| `sim-setup reset` | Archive `.env` and wipe managed data |
175+
| `sim-setup reset` | Archive `.env` and wipe managed data. On Kubernetes it runs `helm uninstall`, which leaves the Postgres volumes behind — they come from a StatefulSet's `volumeClaimTemplates`, which Kubernetes never deletes. Delete those PVCs yourself to reset the data |
176176
| `sim-setup config` | Show configured capabilities and integrations |
177177
| `sim-setup doctor` | Check the setup. `--fix` repairs what it can, `--json` prints machine-readable output |
178178
| `sim-setup add <feature>` | Configure one capability: `email`, `storage`, `sandbox`, `jobs`, `cache`, `knowledge`, `knowledge-embeddings`, `chat`, `llm`, or `integration <slug>` |

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

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -196,10 +196,11 @@ 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-
# 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
199+
# The Ollama service is profile-scoped, so an unprofiled command silently
200+
# skips it. Use the profile you installed with.
201+
PROFILE=gpu # or cpu
202+
docker compose -f docker-compose.ollama.yml --profile "$PROFILE" pull
203+
docker compose -f docker-compose.ollama.yml --profile "$PROFILE" up -d
203204
```
204205

205206
<Callout type="warn">

0 commit comments

Comments
 (0)