gatewayd-io / gatewayd / 13485346862

60%

Build Type

push

github

Committed by web-flow

Commit Message

Add Raft cluster peer management (GetPeers, AddPeer, RemovePeer) (#663)

* feat(raft): add peer discovery and cluster join functionality

Adds support for automatic peer discovery and cluster joining for non-bootstrap nodes.
Key changes:

- Add AddPeer RPC endpoint to allow nodes to join an existing cluster
- Implement TryConnectToCluster() to handle automatic cluster joining
- Forward AddPeer requests to leader if received by follower
- Add protobuf definitions for AddPeer request/response
- Update .gitignore to exclude raft node data files

This change allows new nodes to automatically discover and join an existing cluster
by attempting to connect to configured peers until successful. Non-leader nodes
will forward join requests to the current leader.

* test: add test cases for Raft AddPeer functionality

Add unit tests to verify the AddPeer behavior in both leader and follower nodes:
- Test successful peer addition when node is leader

* test: enhance TestAddPeer to verify peer addition for leader and follower nodes

- Updated TestAddPeer to include checks for adding peers when the node is a leader and a follower.
- Introduced temporary directories for each node to ensure isolated testing environments.
- Added assertions to confirm that both new peers are successfully integrated into the cluster.
- Improved test reliability by implementing a loop to wait for both nodes to join the cluster before completing the test.

* feat(raft): implement RemovePeer functionality and associated tests

- Added RemovePeer RPC endpoint to the Raft service, allowing nodes to remove peers from the cluster.
- Introduced RemovePeerRequest and RemovePeerResponse message types in the protobuf definitions.
- Updated RaftNode to handle peer removal, including forwarding requests to the leader if the node is not the leader.
- Enhanced the README documentation to include details about the new RemovePeerRequest and RemovePeerResponse.
- Implemented unit tests for the RemovePee... (continued)

Run Details

692 of 1121 new or added lines in 13 files covered. (61.73%)

7 existing lines in 4 files now uncovered.

5875 of 9320 relevant lines covered (63.04%)

19.16 hits per line