Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make 'rabbitmqctl rename_cluster_node' and 'rabbitmqctl update_cluster_nodes ' no-ops #10369

Merged
merged 9 commits into from Jan 19, 2024

Conversation

michaelklishin
Copy link
Member

@michaelklishin michaelklishin commented Jan 18, 2024

This makes two commands that rename offline cluster members
no-ops. The commands are really complex under
the hood and are fundamentally incompatible
with a few key Raft-based features:

  • Khepri
  • Quorum queues
  • Streams

Because Khepri first ships in RabbitMQ 3.13,
now is the time to effectively eliminate both.

It will be permanently removed together with
other deprecated CLI commands in 4.0.

Per discussion with the team.

Some background

This command was originally introduces for the needs of the "1.0-ish" TAS (né PCF) RabbitMQ tile around 2015. Due to some BOSH templating changes, the tile could not keep the original node naming scheme and we had to introduce a way of renaming nodes during a cluster upgrade.

Needless to say, it is very complex and entirely Mnesia-specific.

It is not widely used and was never widely recommended in any RabbitMQ or TAS RabbitMQ tile docs.

Further comments

Closes #10367.

@michaelklishin michaelklishin added this to the 3.13.0 milestone Jan 18, 2024
@mergify mergify bot added the bazel label Jan 18, 2024
@michaelklishin michaelklishin changed the title Make 'rabbitmqctl rename_cluster_node' a no-op Make 'rabbitmqctl rename_cluster_node' and 'rabbitmqctl update_cluster_nodes ' no-ops Jan 18, 2024
michaelklishin and others added 8 commits January 19, 2024 11:22
This makes a command that renames cluster members
a no-op. This command is really complex under
the hood and is fundamentally incompatible
with a few key Raft-based features:

 * Khepri
 * Quorum queues
 * Streams

Because Khepri first ships in RabbitMQ 3.13,
now is the time to effectively eliminate this
command.

It will be permanently removed together with
other deprecated CLI commands in 4.0.

Per discussion with the team.

Closes #10367.
@michaelklishin michaelklishin merged commit 7f8f229 into main Jan 19, 2024
17 of 19 checks passed
@michaelklishin michaelklishin deleted the rabbitmq-server-10367 branch January 19, 2024 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Disallow use of the rename_cluster_node command by default or consider making it a no-op
2 participants