Failed to refresh slots cache

[damianhodgkiss]

yep, have read the other issues.. we are using AWS redis CLUSTER and it does work fine after the initial attempts.. our issue is we are seeing Failed to refresh slots cache the first time our scripts run because of a low timeout (we are connecting to multiple redis stores for different purposes which may be contributing to needing a slightly higher timeout)

https://github.com/luin/ioredis/blob/master/lib/cluster/index.js#L585-L612

the slots command being issued has a hardcoded 1,000ms timeout which doesn't seem to be enough in some cases. Modifying the timeout in node_modules... to 10,000ms rids the error but doesn't work for our deployment scenario unless we fork the module so just wondering if we can get a config variable for this timeout?

[shaharmor]

Sound reasonable. Care to make a PR?

[damianhodgkiss]

can do.. any preferences what you'd like the option called so it keeps with your option naming? else maybe just refreshSlotsTimeout (on the main cluster config/options)

[shaharmor]

slotsRefreshTimeout should be good. @luin ?

[luin]

@shaharmor Sounds good to me.

[luin]

Released with v3.1.0 🍻