From 298df2a30bf59cc4010c2d0e93a697e04fff2039 Mon Sep 17 00:00:00 2001 From: "goodolclint-claude[bot]" <323206664+goodolclint-claude[bot]@users.noreply.github.com> Date: Tue, 1 Sep 2026 17:42:07 -0500 Subject: [PATCH] docs: cite the right lines for the fixed VMIDs The review caught that run-integration.sh:106-138 covers pve_vmid() but not STORAGE_VMID, which is set at line 80. Cite both accurately. --- CLAUDE.md | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/CLAUDE.md b/CLAUDE.md index cbe44df..d0693aa 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -158,7 +158,8 @@ unaffected. Copy `tests/.env.test.example` to `tests/.env.test` — it lists every required variable, including the Terraform storage pools CI supplies from repository variables. -**The nested VMIDs are fixed constants** (5080, 5091, 5092 — `run-integration.sh:106-138`) and +**The nested VMIDs are fixed constants** (storage VM 5080 at `run-integration.sh:80`; nodes 5091 +and 5092 in `pve_vmid()` at `:112`) and are shared with CI on the same parent cluster. Never start a local run while a CI integration run is in flight, and always finish with `force-cleanup`: leftover guests fail the next run's headroom guard.