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

Scaling with Redis Cluster #2427

Closed
jiangyunpeng opened this issue Jun 5, 2023 · 4 comments
Closed

Scaling with Redis Cluster #2427

jiangyunpeng opened this issue Jun 5, 2023 · 4 comments

Comments

@jiangyunpeng
Copy link
Contributor

jiangyunpeng commented Jun 5, 2023

hello

I wanted to talk about the 《Remove a node 》sections in this chapter , recently i prepared to remove some failed node in cluster ,when i used redis-cli --cluster del-node 10.20.32.2:6383 44a0355dff74211554c4df6a7daaca5d2cde1227 i got error: " Could not connect to Redis at failed-node-ip:failed-node-port: Connection refused ".

The document did not note how to delete failed node . actually it should use redis-cli --cluster call 10.20.32.3:6380 cluster forget 808bd1a6208b833f005305113fe5d9d5102ede96 i thought cluster forget command must to be applied on every node ( if i was wrong welcome to correct me)

i've spend a few time to find the correct way to remove failed node , so i hope redis-doc will add it , thank you

@itamarhaber
Copy link
Member

Hello @jiangyunpeng - thanks for bringing this up. Would you like to create PR that fixes this?

@jiangyunpeng
Copy link
Contributor Author

sure I'm happy to do some work for the community. i will create PR soon

jiangyunpeng added a commit to jiangyunpeng/redis-doc that referenced this issue Jun 8, 2023
jiangyunpeng added a commit to jiangyunpeng/redis-doc that referenced this issue Jun 8, 2023
@jiangyunpeng
Copy link
Contributor Author

jiangyunpeng commented Jun 8, 2023

hello @itamarhaber there is my PR #2434

zuiderkwast pushed a commit that referenced this issue Dec 15, 2023
`redis-cli --cluster del-node` doesn't work since it tries to connect to all nodes. `--cluster call CLUSTER FORGET` works though.
@zuiderkwast
Copy link
Contributor

Closed by #2434

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

3 participants