]> git.proxmox.com Git - mirror_corosync.git/commit
totemsrp: Fix leave message regression
authorJan Friesse <jfriesse@redhat.com>
Thu, 19 Apr 2018 10:52:39 +0000 (12:52 +0200)
committerJan Friesse <jfriesse@redhat.com>
Mon, 23 Apr 2018 15:46:05 +0000 (17:46 +0200)
commite45bbcc92a1176634ac27f7771c9a5beb01f2791
tree98b5d0de1ce3b6e8ed4b266a9496ac97d859569f
parentdc590159f598b3b6c6a3e0413c3eb02edaa3dc68
totemsrp: Fix leave message regression

Leave message in totem is just join message where leaving member is
excluded from member list and included in fail list. It also contains
special nodeid in header.nodeid and system_from.nodeid fields.

Before "totem: Use nodeid ONLY in srp_addr" fix, most of the functions
were using system_from addresses and not nodeid, which was used only in
one specific case for memb_consensus_set function.

After the patch, addresses are gone and only nodeid is used. Result is,
that leaving node nodeid is not added into local fail list
(my_faillist) so node is unable to reach consensus till token timeout,
which starts new gather process.

Solution is to send valid leaving node nodeid in system_from.nodeid and
handle specific case for memb_consensus_set in memb_join_process.

Signed-off-by: Jan Friesse <jfriesse@redhat.com>
Reviewed-by: Christine Caulfield <ccaulfie@redhat.com>
exec/totemsrp.c