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

Update AccountManager._getNodeClient() to parallel the test connection calls and reduce the total time it takes to fail for load balancer IP logic #1036

Closed
3 tasks done
Tracked by #47
jeromy-cannon opened this issue Dec 30, 2024 · 2 comments · Fixed by #1091
Assignees
Labels
P1 High priority issue. Required to be completed in the assigned milestone. released on @0.33.x released

Comments

@jeromy-cannon
Copy link
Contributor

jeromy-cannon commented Dec 30, 2024

During testing a 10 node configuration Alex believed the process was hanging. Upon examination of the logs, it was using the default LoadBalancer configuration deployed to Latitude, but Latitude was blocking the calls. Each node configuration for the node client was taking 2 minutes. So, it would have been a 20 minute wait every time it had to create a client node, which is unacceptable. He was able to override the values file and set it to ClusterIP and avoid the issue as a workaround.

Tasks

Preview Give feedback
@jeromy-cannon jeromy-cannon changed the title Update AccountManager.testNodeClientConnection() to parallel the test connection calls and reduce the total time it takes to fail for load balancer IP logic Update AccountManager._getNodeClient() to parallel the test connection calls and reduce the total time it takes to fail for load balancer IP logic Dec 30, 2024
@jeromy-cannon jeromy-cannon added the P1 High priority issue. Required to be completed in the assigned milestone. label Dec 30, 2024
@jeromy-cannon jeromy-cannon self-assigned this Dec 30, 2024
@swirlds-automation
Copy link
Contributor

🎉 This issue has been resolved in version 0.33.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@swirlds-automation
Copy link
Contributor

🎉 This issue has been resolved in version 0.33.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 High priority issue. Required to be completed in the assigned milestone. released on @0.33.x released
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants