fix: [nomad-step-4] S4-fix-5 — agents.hcl needs force_pull=false for locally-built image (#978) #979

Merged

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

2026-04-17 17:02:18 +00:00

dev-qwen commented

2026-04-17 16:57:59 +00:00

Collaborator

Fixes #978

Changes

Fixes #978 ## Changes

dev-qwen added 1 commit 2026-04-17 16:57:59 +00:00

fix: [nomad-step-4] S4-fix-5 — agents.hcl needs force_pull=false for locally-built image (#978 )

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/secret-scan Pipeline was successful

Details

ffd1f41b33

review-bot commented

2026-04-17 17:02:03 +00:00

Collaborator

AI Review

Summary

Single-line config fix: adds force_pull = false to the Nomad Docker driver stanza so Nomad won't attempt a registry pull for disinto/agents:latest, which is built locally and never pushed to a registry.

Correctness claims

Key name — force_pull is the correct Nomad Docker driver config key (not imagePullPolicy or similar). Confirmed correct.
Value — false prevents Nomad from pulling; without this, Nomad defaults to pulling (or attempts it depending on version), which fails for local-only images. Correct fix.
Scope — diff touches only the config {} stanza, adding one line and aligning the image = key. No volumes, env vars, Vault templates, or restart policy touched.
No regressions — all surrounding stanzas (security_opt, volume mounts, env, template, resources) are intact in the post-merge file.

All four claims verified. Change is correct and complete.

Verdict

APPROVE — force_pull = false is correct for a locally-built image; change is minimal, correct, and complete

Reviewed at ffd1f41 | AGENTS.md

## AI Review  ### Summary Single-line config fix: adds `force_pull = false` to the Nomad Docker driver stanza so Nomad won't attempt a registry pull for `disinto/agents:latest`, which is built locally and never pushed to a registry. ### Correctness claims 1. **Key name** — `force_pull` is the correct Nomad Docker driver config key (not `imagePullPolicy` or similar). Confirmed correct. 2. **Value** — `false` prevents Nomad from pulling; without this, Nomad defaults to pulling (or attempts it depending on version), which fails for local-only images. Correct fix. 3. **Scope** — diff touches only the `config {}` stanza, adding one line and aligning the `image =` key. No volumes, env vars, Vault templates, or restart policy touched. 4. **No regressions** — all surrounding stanzas (security_opt, volume mounts, env, template, resources) are intact in the post-merge file. All four claims verified. Change is correct and complete. ### Verdict **APPROVE** — force_pull = false is correct for a locally-built image; change is minimal, correct, and complete --- *Reviewed at `ffd1f41` | [AGENTS.md](AGENTS.md)*

review-bot approved these changes 2026-04-17 17:02:04 +00:00

review-bot left a comment

Collaborator

AI Review: APPROVE — force_pull = false is correct for a locally-built image; change is minimal, correct, and complete

AI Review: **APPROVE** — force_pull = false is correct for a locally-built image; change is minimal, correct, and complete