samba.git/ctdb/server/ctdb_server.c, branch talloc-2.4.0

ctdb-daemon: Don't mark a node as unhealthy when connecting to it

2021-09-09T02:38:34+00:00

Remote nodes are already initialised as UNHEALTHY when the node list
is initialised at startup (ctdb_load_nodes_file() calls
convert_node_map_to_list()) and when disconnected (ctdb_node_dead()).
So, drop this code.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14784
Signed-off-by: Martin Schwenke 
Reviewed-by: Amitay Isaacs 

Autobuild-User(master): Amitay Isaacs 
Autobuild-Date(master): Thu Sep  9 02:38:34 UTC 2021 on sn-devel-184

ctdb-daemon: more logical whitespace, debug modernisation

2020-03-12T03:47:30+00:00

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14295

Signed-off-by: Martin Schwenke 
Reviewed-by: Ralph Boehme

ctdb-daemon: ensure restart() callback is called in half-connected state

2020-03-12T03:47:30+00:00

If NODE_FLAGS_DISCONNECTED is set the node can be in half-connected state. With
this change we ensure to restart the transport for this case.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14295

Signed-off-by: Ralph Boehme 
Reviewed-by: Martin Schwenke

ctdb-daemon: Avoid memory leak when packet is deferred

2019-10-24T04:06:43+00:00

Signed-off-by: Amitay Isaacs 
Reviewed-by: Martin Schwenke

ctdb-daemon: Add function ctdb_ip_to_node()

2019-08-16T21:30:35+00:00

This is the core logic from ctdb_ip_to_pnn(), so re-implement that
that function using ctdb_ip_to_node().

Something similar (ctdb_ip_to_nodeid()) was recently removed in commit
010c1d77cd7e192b1fff39b7b91fccbdbbf4a786 because it wasn't required.
Now there is a use case.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14084

Signed-off-by: Martin Schwenke 
Reviewed-by: Amitay Isaacs

ctdb-daemon: Replace function ctdb_ip_to_nodeid() with ctdb_ip_to_pnn()

2019-07-05T05:03:23+00:00

Node ID is a poorly defined concept, indicating the slot in the node
map where the IP address was found.  This signed value also ends up
compared to num_nodes, which is unsigned, producing unwanted warnings.

Just return the PNN because this what both callers really want.

Signed-off-by: Martin Schwenke 
Reviewed-by: Amitay Isaacs

ctdb-daemon: Fix signed/unsigned comparisons by declaring as unsigned

2019-07-05T05:03:23+00:00

Signed-off-by: Martin Schwenke 
Reviewed-by: Amitay Isaacs

ctdb-server: Rename CTDB_BROADCAST_VNNMAP -> CTDB_BROADCAST_ACTIVE

2018-07-02T06:51:22+00:00

This broadcast is misnamed.  Both places where this type of broadcast
is used expect the broadcast to go to all active nodes.

Make the corresponding change to the semantics in the daemon by
sending to all active nodes.

There is a mismatch between the ideas of VNN map and active nodes.  A
node that is not in the VNN map but is active can still host database
records.  These were the same until the LMASTER capability was
introduced and then the logic was not updated.

The only place where the VNN map is relevant is when finding the
location master of a record in the migration code.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=13499

Signed-off-by: Martin Schwenke 
Reviewed-by: Amitay Isaacs

ctdb-daemon: Add code to process ctdb_req_tunnel packets

2017-10-10T09:45:19+00:00

Signed-off-by: Amitay Isaacs 
Reviewed-by: Martin Schwenke

ctdb-daemon: Increase priority of logs for node connect/disconnect

2017-07-04T11:11:16+00:00

Signed-off-by: Amitay Isaacs 
Reviewed-by: Martin Schwenke