-
Notifications
You must be signed in to change notification settings - Fork 41
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
Better handling of master failover #238
Comments
Hmm, This makes sense but
How can we do this? Considering we don't have a proxy to do different replies. It's upto the older pod to return READONLY (which they will once they are configured as a replica). wdyt? 🤔 |
I think it would be fine to issue TCP connection reset for every open connection if pod loses write master status. You can make this behaviour optional via command line flag or something like that |
Any updates regarding this problem? |
Functionality to kill client connections currently does not exist in Dragonfly (See dragonflydb/dragonfly#2178). and there is no common Kubernetes abstraciton/functionality to do this. :/ cc: @romange |
Hello! I we are trying to do some tests with failover and wondering the best way to do this. Don't want to delete the master pod, as that always works. Want to issue a failover, but
And when I try manually
on replica works, but then when I try to get the old master to follow new one this happens.
Any ideas? |
found reference to a command called REPLTAKEOVER, but does not seem like I can get it to do anything, help is also empty
|
It looks like most applications do have Redis reconnect support if Redis operation results in error such as
ReplyError: READONLY You can't write against a read only replica
. In fact the ioredis docs specifically mention this usecase.This is related to #149 (comment)
To better support dumb clients I would advise implementing TCP disconnect when Dragonfly pod loses master role to ensure clients reconnect to the new Dragonfly master pod
The text was updated successfully, but these errors were encountered: