]> git.proxmox.com Git - pve-ha-manager.git/commit
lrm: also check CRM node-status for determining fence-request
authorThomas Lamprecht <t.lamprecht@proxmox.com>
Mon, 17 Jan 2022 14:52:53 +0000 (15:52 +0100)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Wed, 19 Jan 2022 12:48:57 +0000 (13:48 +0100)
commit30fc7ceedb7f3047659f22d063cc16c94c20dd7a
tree8f9011cbbde555e2e2a9a0b930f5dab3faa587c1
parent303490d8f11cb51601086c53fc53ac7cca047ffa
lrm: also check CRM node-status for determining fence-request

This fixes point 2. of commit 3addeeb - avoiding that a LRM goes
active as long as the CRM still has it in (pending) `fence` state,
which can happen after a watchdog reset + fast boot. This avoids that
we interfere with the CRM acquiring the lock, which is all the more
important once a future commit gets added that ensures a node isn't
stuck in `fence` state if there's no service configured (anymore) due
to admin manually removing them during fencing.

We explicitly fix the startup first to better show how it works in
the test framework, but as the test/sim hardware can now delay the
CRM now while keeping LRM running, the second test (i.e.,
test-service-command9) should still trigger after the next commit, if
this one would be reverted or broken otherwise.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
src/PVE/HA/LRM.pm
src/test/test-service-command8/log.expect
src/test/test-service-command9/log.expect