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

Add self node in first cluster node response #107

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nidhi-ag
Copy link

@nidhi-ag nidhi-ag commented May 13, 2020

Recently we faced an issue on production where predixy was throwing "MOVED" error just after the process started. After debugging found that

"Info" command output - Just after predixy got healthy (health check using ping command)
Server:prod-adserver.xoptuc.clustercfg.apse1.cache.amazonaws.com:6379
Role:master
Group:abbff3be959032ba777a295f9d4210a281841d51
DC:
CurrentIsFail:0
Connections:5
Connect:5
Requests:249
Responses:249
SendBytes:25169
RecvBytes:4109

Server:192.168.5.176:6379
Role:slave
Group:abbff3be959032ba777a295f9d4210a281841d51
DC:
CurrentIsFail:0
Connections:5
Connect:5
Requests:47
Responses:47
SendBytes:2072

And after 10-30 seconds (refresh interval is 30) after predixy got healthy
Server:prod-adserver.xoptuc.clustercfg.apse1.cache.amazonaws.com:6379
Role:master
Group:abbff3be959032ba777a295f9d4210a281841d51
DC:
CurrentIsFail:0
Connections:5
Connect:5
Requests:249
Responses:249
SendBytes:25169
RecvBytes:4109

Server:192.168.5.176:6379
Role:slave
Group:abbff3be959032ba777a295f9d4210a281841d51
DC:
CurrentIsFail:0
Connections:5
Connect:5
Requests:47
Responses:47
SendBytes:2072
RecvBytes:962

Server:192.168.5.46:6379
Role:master
Group:abbff3be959032ba777a295f9d4210a281841d51
DC:
CurrentIsFail:0
Connections:5
Connect:5
Requests:190
Responses:125
SendBytes:10775
RecvBytes:1409

After checking the code and debug logs found that "self" replica is not getting added in the pool in the first attempt. It gets added in the second call of "cluster nodes".

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

Successfully merging this pull request may close these issues.

1 participant