DOCFIX-188 followup: correct the pass criteria (anchor on virsh dominfo, not the QEMU -m string)

Fork: 0

OpenStack / openstack-caracal-dc-dc

Browse code DOCFIX-188 followup: correct the pass criteria (anchor on virsh dominfo, not the QEMU -m string) libvirt normalizes MiB to KiB in the QEMU cmdline, so '-m size=2097152k' would be SUCCESS, not failure -- the original criterion would have read a correct boot as a failed one. Anchor on 'virsh dominfo' Max memory, which is unambiguous. Also: re-run plan AFTER the undefine (the captured in-place plan is not what executes), and pre-stage the two fallbacks (provider ignores memory_unit -> *1024; boots but no router -> sizing, not a regression). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Y117t1F525ba9r1vXSBETk dc-dc-stage1-phase0-first-exec
1 parent 6ad637f commit b7390d916c02146174a4ca42a2136cd4829334b3 JANeumatrix authored 1 day ago

Browse code

libvirt normalizes MiB to KiB in the QEMU cmdline, so '-m size=2097152k' would be
SUCCESS, not failure -- the original criterion would have read a correct boot as a
failed one. Anchor on 'virsh dominfo' Max memory, which is unambiguous. Also: re-run
plan AFTER the undefine (the captured in-place plan is not what executes), and
pre-stage the two fallbacks (provider ignores memory_unit -> *1024; boots but no
router -> sizing, not a regression).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Y117t1F525ba9r1vXSBETk

dc-dc-stage1-phase0-first-exec

1 parent 6ad637f commit b7390d916c02146174a4ca42a2136cd4829334b3

JANeumatrix authored 1 day ago

Patch

Unified Split

Showing 2 changed files

Ignore Space Show notes View docs/changelog-20260712-libvirt-memory-unit-rootcause.md

Ignore Space Show notes View docs/session-ledger.md

Show line notes below