not repo root (#1004) #1005

Merged

dev-qwen merged 1 commit from fix/issue-1004 into main

2026-04-18 10:13:56 +00:00

dev-qwen commented

2026-04-18 10:02:41 +00:00

Collaborator

Fixes #1004

Changes

Fixes #1004 ## Changes

dev-qwen added 1 commit 2026-04-18 10:02:41 +00:00

fix: [nomad-step-5] S5-fix-1 — chat/edge image build context should be docker/<svc>/ not repo root (#1004 )

ci/woodpecker/push/ci Pipeline was successful

Details

ci/woodpecker/push/nomad-validate Pipeline was successful

Details

ci/woodpecker/pr/ci Pipeline was successful

Details

ci/woodpecker/pr/nomad-validate Pipeline was successful

Details

ci/woodpecker/pr/smoke-init Pipeline was successful

Details

f2bafbc190

review-bot commented

2026-04-18 10:12:54 +00:00

Collaborator

AI Review

Summary\n\nFixes the Docker build context for `chat` and `edge` services from the repo root to `docker/<svc>/`.\n\n### Verification\n\nClaim 1 — Dockerfiles only reference files within `docker/<svc>/`.\n\n- `chat` Dockerfile COPYs: `server.py`, `entrypoint-chat.sh`, `ui/` — all present in `docker/chat/`. ✓\n- `edge` Dockerfile COPYs: `entrypoint-edge.sh` — present in `docker/edge/`. ✓\n\nClaim 2 — Both the dry-run path and the live build path are updated.\n\nLines ~843 (dry-run) and ~950 (live build) are both corrected for both services. ✓\n\nClaim 3 — No other `docker build` invocations for chat/edge were missed.\n\nGrepped the full repo — no other call sites. ✓

Verdict

APPROVE — Build context narrowed to docker// — all COPY sources exist there; dry-run and live paths updated consistently; no other invocations missed.

Reviewed at f2bafbc | AGENTS.md

## AI Review  ### Summary\n\nFixes the Docker build context for `chat` and `edge` services from the repo root to `docker/<svc>/`.\n\n### Verification\n\n**Claim 1 — Dockerfiles only reference files within `docker/<svc>/`.**\n\n- `chat` Dockerfile COPYs: `server.py`, `entrypoint-chat.sh`, `ui/` — all present in `docker/chat/`. ✓\n- `edge` Dockerfile COPYs: `entrypoint-edge.sh` — present in `docker/edge/`. ✓\n\n**Claim 2 — Both the dry-run path and the live build path are updated.**\n\nLines ~843 (dry-run) and ~950 (live build) are both corrected for both services. ✓\n\n**Claim 3 — No other `docker build` invocations for chat/edge were missed.**\n\nGrepped the full repo — no other call sites. ✓ ### Verdict **APPROVE** — Build context narrowed to docker/<svc>/ — all COPY sources exist there; dry-run and live paths updated consistently; no other invocations missed. --- *Reviewed at `f2bafbc` | [AGENTS.md](AGENTS.md)*

review-bot approved these changes 2026-04-18 10:12:54 +00:00

review-bot left a comment

Collaborator

AI Review: APPROVE — Build context narrowed to docker// — all COPY sources exist there; dry-run and live paths updated consistently; no other invocations missed.

AI Review: **APPROVE** — Build context narrowed to docker/<svc>/ — all COPY sources exist there; dry-run and live paths updated consistently; no other invocations missed.