Skip to content

Conversation

@enjoy-binbin
Copy link
Contributor

@enjoy-binbin enjoy-binbin commented Feb 16, 2022

I see that the bus-port parameter is listed in cluster help, but not documented in redis-doc

redis/src/cluster.c

Lines 5022 to 5023 in 3881f78

"MEET <ip> <port> [<bus-port>]",
" Connect nodes into a working cluster.",

In CLUSTER MEET, bus-port argument was added in 11436b1.
For cluster announce ip / port implementation, part of the
4.0-RC1.

And in #9389, we add a new cluster-port config and make
cluster bus port configurable, part of the 7.0-RC1.

In `CLUSTER MEET`, bus-port argument was added in 11436b1.
For cluster announce ip / port implementation, part of the
4.0-RC1.

And in redis#9389, we add a new cluster-port config and make
cluster bus port configurable, part of the 7.0-RC1.
@enjoy-binbin enjoy-binbin changed the title Add cport argument in cluster-meet.json Add bus_port argument in cluster-meet.json Feb 27, 2022
@enjoy-binbin enjoy-binbin added the state:to-be-merged The PR should be merged soon, even if not yet ready, this is used so that it won't be forgotten label Feb 27, 2022
@enjoy-binbin enjoy-binbin requested review from madolson and removed request for madolson April 25, 2022 02:09
@oranagra oranagra merged commit aabce89 into redis:unstable Jun 23, 2022
@enjoy-binbin enjoy-binbin deleted the cluster_meet_cport branch June 23, 2022 05:03
enjoy-binbin added a commit to enjoy-binbin/redis that referenced this pull request Jul 31, 2023
In `CLUSTER MEET`, bus-port argument was added in 11436b1.
For cluster announce ip / port implementation, part of the
4.0-RC1.

And in redis#9389, we add a new cluster-port config and make
cluster bus port configurable, part of the 7.0-RC1.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state:to-be-merged The PR should be merged soon, even if not yet ready, this is used so that it won't be forgotten

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants