fix(self-host): repair dev compose generation, kitchen-sink serverful runner, and postgres/udb bootstrap

[MasterPtato]

No description provided.

[MasterPtato]

Stack for rivet-dev/rivet

Current stack:

• [SLOP(claude-opus-4-8-high)] perf(universaldb): batch leader apply and fold follower commit round-trips #5342 vtrtqpyp · 2026-06-26 02:20:21
• [slopfix] docs(self-hosting): promote postgres from experimental to recommended OSS multi-node backend #5341 zsmysquk · 2026-06-26 02:20:19
• [slopfix] fix(self-host): repair dev compose generation, kitchen-sink serverful runner, and postgres/udb bootstrap #5340 wpvkrymu · 2026-06-25 22:53:05 👈
• [SLOP(claude-opus-4-8-high)] feat(universaldb): graceful postgres leader handoff on shutdown #5337 mvxpzxsl · 2026-06-25 20:34:04
• [slopfix] test(universaldb): postgres leader failover + abort resolver task on driver drop #5336 tvmntvqo · 2026-06-25 20:34:03
• [SLOP(claude-opus-4-8-high)] feat(ups): table-backed postgres transport with coalesced doorbell #5335 xutxrzrv · 2026-06-25 20:34:01
• [SLOP(claude-opus-4-8-high)] feat(universaldb): postgres leader-resolver driver overhaul #5329 xrrmomwz · 2026-06-25 00:55:07
• [SLOP(claude-opus-4-8)] feat(pegboard-envoy): rate-limit envoy ws ingress and cap get_pages, trim unused metrics #5327 tnzyzqrl · 2026-06-24 20:57:36
• [SLOP(claude-opus-4-8)] feat(util): add rate limiter primitive and ingress throttles for actor create and gateway websocket #5326 pnyswwtm · 2026-06-24 20:57:35
• [SLOP(claude-opus-4-8)] feat(rivetkit-core): add serde duration and size metrics #5322 nqkrytvr · 2026-06-23 22:49:13
• Add observation for serde_bare #5321 oskoutmw · 2026-06-23 20:23:10
• Add observation and ser/de metrics for serde calls #5320 ztnuwvts · 2026-06-23 20:23:09

Dependencies:

• [SLOP(claude-opus-4-8-medium)] fix(rivetkit-napi): reap completed tasks from event-loop JoinSet (#5315) #5314 xxqwtrzp · 2026-06-22 22:42:01
• main

Get stack: forklift get 5340
Push local edits: forklift submit
Merge when ready: forklift merge 5340

[claude]

PR Review

Overall this is a solid infrastructure cleanup. The build context path correction (../../.. → ../..), structural rename from self-host/compose/{variant}/ to self-host/{variant}/, and the runner-config-init approach to bootstrapping the no_runner_config dependency are all sensible. A few items worth flagging:

---

Breaking change: metrics namespace prefix

engine/packages/metrics/src/registry.rs — changing None to Some("rivet".to_string()) prepends rivet_ to every metric name in the engine. Any existing Prometheus rules, Grafana dashboards, or alerting queries that reference metric names directly will silently break. If this prefix was missing intentionally or these consumers don't exist yet, fine — but the change should be noted in the PR description so operators of existing installs know to update their dashboards.

---

Breaking migration: postgres volume path

Changing postgres-data:/var/lib/postgresql/data → postgres-data:/var/lib/postgresql means existing named volumes that stored data under the /data subdirectory will appear empty after pulling this change. Anyone with an existing dev environment will silently lose their local state. This is likely acceptable for a dev environment, but a one-line migration note (e.g. docker volume rm <project>_postgres-data) in the PR description would help.

---

Non-deterministic image pin in runner-config-init

All compose files add:

image: curlimages/curl:latest

Using latest is non-deterministic. This is probably fine for a dev self-host workflow, but a pinned tag (e.g. curlimages/curl:8.12.1) would make the setup more reproducible across machines and time. Low priority, but worth tracking.

---

Multi-DC runner-config-init dependency

In dev-multidc and dev-multidc-multinode, the init container only depends_on: rivet-engine-dc-a, yet runners in dc-b and dc-c also depend on it completing. This is functionally correct if the single PUT /runner-configs/... call to dc-a propagates the config to all engines (since the payload includes all DCs). If that propagation ever becomes async or conditional, dc-b/dc-c runners could start before their config is visible. Worth a brief comment in the compose file explaining the assumption.

---

CLAUDE.md reference is now stale

CLAUDE.md says:

Do not edit self-host/compose/dev* configs directly. Edit the template in self-host/compose/template/ and rerun.

After this PR the generated files live at self-host/{variant}/ (not self-host/compose/{variant}/). The instruction's path reference should be updated to self-host/dev*/ to stay accurate.

---

Minor: prod-file-system removal

The prod-file-system compose variant is deleted with no explanation in the PR description. If this was a published/documented variant, a note about what replaces it (or that it's simply retired) would help future readers of the git log.

---

Overall the mechanical work here looks correct — the build context fix, the structural rename, the serverful kitchen-sink pivot, and the postgres max_connections=500 bump are all reasonable. The metrics prefix and postgres volume changes are the ones most likely to affect existing operators and deserve a callout.