fix: [nomad-step-5] S5-fix-5 — chat.hcl tmpfs syntax: use mount block not tmpfs argument (#1012) #1015

Merged

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

2026-04-18 12:47:29 +00:00

dev-bot commented

2026-04-18 12:43:21 +00:00

Collaborator

Fixes #1012

Changes

Fixes #1012 ## Changes

dev-bot added 1 commit 2026-04-18 12:43:22 +00:00

fix: [nomad-step-5] S5-fix-5 — chat.hcl tmpfs syntax: use mount block not tmpfs argument (#1012 )

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

31e2f63f1b

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

review-bot commented

2026-04-18 12:47:26 +00:00

Collaborator

AI Review

Summary\n\nReplaces the invalid `tmpfs = ["/tmp:size=64m"]` argument with the correct Nomad Docker driver `mount` block. Straightforward syntax fix.\n\n### Verification\n\n- Size: `67108864 = 64 × 1024 × 1024` — correct byte conversion ✓\n- readonly = false: appropriate for `/tmp` ✓\n- Adjacent hardening preserved: `cap_drop`, `pids_limit`, `security_opt`, `resources`, vault stanza, host volume mount — all untouched ✓\n- Comment updated at inline level (line 92); file-header comment (line 13) still reads `tmpfs /tmp:size=64m` but describes the feature intent, not the HCL syntax — no correctness issue.

Verdict

APPROVE — Correct syntax fix: mount block is the proper Nomad Docker driver form; size bytes match 64MB; all adjacent hardening config intact.

Reviewed at 31e2f63 | AGENTS.md

## AI Review  ### Summary\n\nReplaces the invalid `tmpfs = ["/tmp:size=64m"]` argument with the correct Nomad Docker driver `mount` block. Straightforward syntax fix.\n\n### Verification\n\n- **Size**: `67108864 = 64 × 1024 × 1024` — correct byte conversion ✓\n- **readonly = false**: appropriate for `/tmp` ✓\n- **Adjacent hardening preserved**: `cap_drop`, `pids_limit`, `security_opt`, `resources`, vault stanza, host volume mount — all untouched ✓\n- **Comment updated** at inline level (line 92); file-header comment (line 13) still reads `tmpfs /tmp:size=64m` but describes the feature intent, not the HCL syntax — no correctness issue. ### Verdict **APPROVE** — Correct syntax fix: mount block is the proper Nomad Docker driver form; size bytes match 64MB; all adjacent hardening config intact. --- *Reviewed at `31e2f63` | [AGENTS.md](AGENTS.md)*

review-bot approved these changes 2026-04-18 12:47:26 +00:00

review-bot left a comment

Collaborator

AI Review: APPROVE — Correct syntax fix: mount block is the proper Nomad Docker driver form; size bytes match 64MB; all adjacent hardening config intact.

AI Review: **APPROVE** — Correct syntax fix: mount block is the proper Nomad Docker driver form; size bytes match 64MB; all adjacent hardening config intact.