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

Horde.NodeListener.set_members times out when adding many nodes at once #227

Open
aloukissas opened this issue Apr 13, 2021 · 4 comments
Open

Comments

@aloukissas
Copy link

I'm running my app on k8s and I notice when I bump the number of pods from e.g. 4 to 20, the call to Horde.NodeListener.set_members times out with the default GenServer.call/3 timeout of 5000 msec. This in turn kills the server and causes the pod to recycle, which starts a vicious cycle. I currently avoid this by slowly increasing the number of pods (e.g. by 2 at a time).

@aloukissas aloukissas changed the title Horde.NodeListener.set_members times out when adding many members at once Horde.NodeListener.set_members times out when adding many nodes at once Apr 13, 2021
@keatz55
Copy link

keatz55 commented May 17, 2021

@aloukissas We are running into a similar issue. Did you find a workaround or do you still have your k8s rollout limited to 2 pods at a time?

@derekkraan
Copy link
Owner

There's a fix coming for this (eventually). I need to make it possible (in DeltaCRDT) to make multiple mutations in one call; the problem right now is that it's having to add each new host one by one, which takes too long if you're going from 2 to 20 nodes.

@derekkraan
Copy link
Owner

Hi @aloukissas and @keatz55 can you please test with 0.8.4 (just released) and let me know if this fixes the problems you're facing with bumping number of nodes suddenly? Registry performance should also be improved (when nodes go down).

@RealVidy
Copy link

@derekkraan I have just tested with 0.8.4 and I get this timeout when starting 2 nodes concurrently on a slow machine.

Setting the timeout value to :infinity did "solve" the issue, of course.

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

4 participants