4 services fail to build in dev container #29

New issue

Closed

opened 2026-02-26 13:21:04 +00:00 by mik-tf · 1 comment

mik-tf commented

2026-02-26 13:21:04 +00:00

Owner

Context

The dev container (PR #28) successfully builds and runs 8 of 12 user-profile services. The remaining 4 fail due to upstream repo issues, not container infrastructure.

Failures

Service	Repo	Exit code	Root cause
hero_indexer	`lhumina_code/hero_index_server`	128	Repo does not exist or deploy key has no access
hero_embedder_openrpc	`lhumina_code/hero_embedder`	2	`make install` fails — cargo build error
hero_voice_openrpc	`lhumina_code/hero_voice`	2	`make install` fails — cargo build error
hero_inspector_openrpc	`lhumina_code/hero_inspector`	2	Default branch is `main` but TOML references `development` — clone likely succeeds but build fails

What needs to happen

hero_indexer: create lhumina_code/hero_index_server repo, or update services/user/hero_indexer.toml to point to the correct repo
hero_embedder: fix make install on the development branch of lhumina_code/hero_embedder
hero_voice: fix make install on the development branch of lhumina_code/hero_voice
hero_inspector: either create a development branch on lhumina_code/hero_inspector, or update services/user/hero_inspector_openrpc.toml to use main

Reproducing

docker run --rm -it \
  -v ~/.ssh/id_ed25519:/root/.ssh/id_ed25519:ro \
  -p 6666:6666 -p 3388:3388 -p 3875:3875 \
  forge.ourworld.tf/lhumina_code/hero_zero:dev

Watch logs for process exited ... state=failed lines.

## Context The dev container (PR #28) successfully builds and runs 8 of 12 user-profile services. The remaining 4 fail due to upstream repo issues, not container infrastructure. ## Failures | Service | Repo | Exit code | Root cause | |---|---|---|---| | hero_indexer | `lhumina_code/hero_index_server` | 128 | Repo does not exist or deploy key has no access | | hero_embedder_openrpc | `lhumina_code/hero_embedder` | 2 | `make install` fails — cargo build error | | hero_voice_openrpc | `lhumina_code/hero_voice` | 2 | `make install` fails — cargo build error | | hero_inspector_openrpc | `lhumina_code/hero_inspector` | 2 | Default branch is `main` but TOML references `development` — clone likely succeeds but build fails | ## What needs to happen 1. **hero_indexer**: create `lhumina_code/hero_index_server` repo, or update `services/user/hero_indexer.toml` to point to the correct repo 2. **hero_embedder**: fix `make install` on the `development` branch of `lhumina_code/hero_embedder` 3. **hero_voice**: fix `make install` on the `development` branch of `lhumina_code/hero_voice` 4. **hero_inspector**: either create a `development` branch on `lhumina_code/hero_inspector`, or update `services/user/hero_inspector_openrpc.toml` to use `main` ## Reproducing ```bash docker run --rm -it \ -v ~/.ssh/id_ed25519:/root/.ssh/id_ed25519:ro \ -p 6666:6666 -p 3388:3388 -p 3875:3875 \ forge.ourworld.tf/lhumina_code/hero_zero:dev ``` Watch logs for `process exited ... state=failed` lines.

mik-tf referenced this issue

2026-02-26 13:21:19 +00:00

Production container with pre-baked service binaries #30

mik-tf referenced this issue from a pull request that will close it,

2026-02-26 14:07:15 +00:00

Fix hero_indexer repo URL in service TOML #31

mik-tf referenced this issue from lhumina_code/hero_inspector

2026-02-26 14:08:23 +00:00

Fix Makefile TARGET detection for container builds #2

mik-tf commented

2026-02-26 14:08:39 +00:00

Author

Owner

Update

The development branch Dockerfile has been restructured — builder stage now uses rust:bookworm which has g++ and python3. This changes which fixes are still needed:

Service	Original issue	Status	Fix
hero_indexer	Wrong repo URL	Fixed — PR #31	TOML: `hero_index_server` → `hero_indexer`
hero_embedder	Missing `g++`	No longer needed	Builder image has `g++`
hero_voice	Missing `libstdc++`	No longer needed	Builder image has `g++`
hero_inspector	Makefile TARGET detection	Fixed — hero_inspector PR #2	Respect `CARGO_TARGET_DIR` directly

PRs

hero_services PR #31 — fix hero_indexer repo URL
hero_inspector PR #2 — fix Makefile TARGET detection

## Update The `development` branch Dockerfile has been restructured — builder stage now uses `rust:bookworm` which has `g++` and `python3`. This changes which fixes are still needed: | Service | Original issue | Status | Fix | |---|---|---|---| | hero_indexer | Wrong repo URL | **Fixed** — PR #31 | TOML: `hero_index_server` → `hero_indexer` | | hero_embedder | Missing `g++` | **No longer needed** | Builder image has `g++` | | hero_voice | Missing `libstdc++` | **No longer needed** | Builder image has `g++` | | hero_inspector | Makefile TARGET detection | **Fixed** — hero_inspector PR #2 | Respect `CARGO_TARGET_DIR` directly | ### PRs - hero_services PR #31 — fix hero_indexer repo URL - [hero_inspector PR #2](https://forge.ourworld.tf/lhumina_code/hero_inspector/pulls/2) — fix Makefile TARGET detection

~~mik-tf referenced this issue 2026-02-26 14:43:52 +00:00~~

Fix container build — correct binary names, CI pipeline, entrypoint #32

mik-tf referenced this issue from a commit

2026-02-26 17:56:45 +00:00

fix: split hero_indexer into openrpc/http to match repo binary names

mik-tf referenced this issue from a pull request that will close it,

2026-02-27 17:22:35 +00:00

Combined deploy — all PRs merged + Dockerfile.prod + TFGrid deployment #43

mik-tf referenced this issue from a commit

2026-03-07 20:11:45 +00:00

Switch Dockerfile zinit server build to development branch

mik-tf closed this issue