ACK
Reviewed-By: Christine Caulfield <ccaulfie@xxxxxxxxxx>
On 02/03/12 11:02, Fabio M. Di Nitto wrote:
From: "Fabio M. Di Nitto"<fdinitto@xxxxxxxxxx>
cmap changes are local to the node only and should not be broadcasted
as configuration changes.
if any change has happened to us, we will inform other nodes via
send_nodeinfo.
Signed-off-by: Fabio M. Di Nitto<fdinitto@xxxxxxxxxx>
---
exec/votequorum.c | 21 +++------------------
1 files changed, 3 insertions(+), 18 deletions(-)
diff --git a/exec/votequorum.c b/exec/votequorum.c
index 9209bdd..5a9b821 100644
--- a/exec/votequorum.c
+++ b/exec/votequorum.c
@@ -1093,33 +1093,18 @@ static void votequorum_refresh_config(
struct icmap_notify_value old_val,
void *user_data)
{
- unsigned int old_votes;
- unsigned int old_expected;
- unsigned int old_qdevice_votes;
-
ENTER();
- old_votes = us->votes;
- old_expected = us->expected_votes;
- old_qdevice_votes = qdevice->votes;
-
/*
* Reload the configuration
*/
votequorum_readconfig(VOTEQUORUM_READCONFIG_RUNTIME);
/*
- * Check for fundamental changes that we need to propogate
+ * activate new config
*/
- if (old_votes != us->votes) {
- votequorum_exec_send_reconfigure(VOTEQUORUM_RECONFIG_PARAM_NODE_VOTES, us->node_id, us->votes);
- }
- if (old_expected != us->expected_votes) {
- votequorum_exec_send_reconfigure(VOTEQUORUM_RECONFIG_PARAM_EXPECTED_VOTES, us->node_id, us->expected_votes);
- }
- if (old_qdevice_votes != qdevice->votes) {
- votequorum_exec_send_reconfigure(VOTEQUORUM_RECONFIG_PARAM_NODE_VOTES, NODEID_QDEVICE, qdevice->votes);
- }
+ votequorum_exec_send_nodeinfo(us->node_id);
+ votequorum_exec_send_nodeinfo(NODEID_QDEVICE);
LEAVE();
}
_______________________________________________
discuss mailing list
discuss@xxxxxxxxxxxx
http://lists.corosync.org/mailman/listinfo/discuss
