---
** [tickets:#2946] amfd: set node_sync_window_closed when timer expires**
**Status:** accepted
**Milestone:** 5.18.12
**Created:** Thu Oct 25, 2018 10:36 AM UTC by Gary Lee
**Last Updated:** Thu Oct 25, 2018 10:36 AM UTC
**Owner:** Gary Lee
When testing split network partitions, sometimes AMFD gets into a loop
processing messages below. Eventually it is aborted by the watchdog.
2018-10-25 16:41:43.051 SC-1 osafamfd[272]: NO Receive message with event
type:13, msg_type:32, from node:2030f, msg_id:0
2018-10-25 16:41:43.052 SC-1 osafamfd[272]: NO Receive message with event
type:12, msg_type:31, from node:2030f, msg_id:0
2018-10-25 16:41:43.052 SC-1 osafamfd[272]: NO Receive message with event
type:13, msg_type:32, from node:2030f, msg_id:0
2018-10-25 16:41:43.052 SC-1 osafamfd[272]: NO Receive message with event
type:12, msg_type:31, from node:2030f, msg_id:0
2018-10-25 16:41:43.052 SC-1 osafamfd[272]: NO Receive message with event
type:13, msg_type:32, from node:2030f, msg_id:0
2018-10-25 16:41:43.052 SC-1 osafamfd[272]: NO Receive message with event
type:12, msg_type:31, from node:2030f, msg_id:0
2018-10-25 16:41:43.053 SC-1 osafamfd[272]: NO Receive message with event
type:13, msg_type:32, from node:2030f, msg_id:0
2018-10-25 16:41:43.053 SC-1 osafamfd[272]: NO Receive message with event
type:12, msg_type:31, from node:2030f, msg_id:0
2018-10-25 16:41:43.053 SC-1 osafamfd[272]: NO Receive message with event
type:13, msg_type:32, from node:2030f, msg_id:0
2018-10-25 16:41:43.053 SC-1 osafamfd[272]: NO Receive message with event
type:12, msg_type:31, from node:2030f, msg_id:0
2018-10-25 16:41:43.054 SC-1 osafamfd[272]: NO Receive message with event
type:13, msg_type:32, from node:2030f, msg_id:0
This fixes it. AMF didn't expect any more sync messages after all nodes have
synced. But a PL split from the main network partition will also send a
headless sync msg.
diff --git a/src/amf/amfd/ndfsm.cc b/src/amf/amfd/ndfsm.cc
index c460d8f..1bc6ed9 100644
--- a/src/amf/amfd/ndfsm.cc
+++ b/src/amf/amfd/ndfsm.cc
@@ -357,6 +357,7 @@ void avd_node_up_evh(AVD_CL_CB *cb, AVD_EVT *evt) {
if (cb->node_sync_tmr.is_active) {
avd_stop_tmr(cb, &cb->node_sync_tmr);
TRACE("stop NodeSync timer");
+ cb->node_sync_window_closed = true;
}
cb->all_nodes_synced = true;
LOG_NO("Received node_up_msg from all nodes");
---
Sent from sourceforge.net because opensaf-tickets@lists.sourceforge.net is
subscribed to https://sourceforge.net/p/opensaf/tickets/
To unsubscribe from further messages, a project admin can change settings at
https://sourceforge.net/p/opensaf/admin/tickets/options. Or, if this is a
mailing list, you can unsubscribe from the mailing list._______________________________________________
Opensaf-tickets mailing list
Opensaf-tickets@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-tickets
