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

Confusing error - setting location before replace #9

Open
martinsumner opened this issue Dec 31, 2024 · 1 comment
Open

Confusing error - setting location before replace #9

martinsumner opened this issue Dec 31, 2024 · 1 comment

Comments

@martinsumner
Copy link
Contributor

When running a replace of an existing node, the new node should first be joined. However, it is not possible to set the location before the replace command - if the replace is prompted after the location has been set, the following error is returned:

riak admin cluster replace [email protected] [email protected]
Failed: '[email protected]' is not a valid replacement candidate.
Only newly joining nodes can be used for replacement.
error

The setting of location is considered a change, and so as there is a change to perform between the join and the replace - the joining node is then not considered to be "newly joined" even though the join has not yet been planned or committed.

When using locations, and replacing a node - do not set the location prior to the replace. This will cause the expected behaviour of all vnodes simply being transferred from the replaced not to the joined node - regardless of location.

If the new node is the same location as the old, setting the location after the replace has completed, should lead to a null cluster change. If the new node is in a different location, and the change is permanent - then a separate cluster change to set the location (and reshuffle the vnodes if necessary) should be staged once the replace is complete.

@martinsumner
Copy link
Contributor Author

Raised as an issue - a clearer error should be returned, and the expected process should be documented.

@tburghart tburghart transferred this issue from OpenRiak/riak_core-forked Feb 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant