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

#865 Move the route table sort into NewTable so that it only happens once. #867

Merged

Conversation

ddreier
Copy link
Contributor

@ddreier ddreier commented Apr 6, 2022

Addresses #865 where route table updates take multiple minutes with tens of thousands of routes.

@CLAassistant
Copy link

CLAassistant commented Apr 6, 2022

CLA assistant check
All committers have signed the CLA.

@ddreier ddreier force-pushed the issue-865-consul-slow-updates branch from cdf27fe to c799fb3 Compare April 6, 2022 23:22
Copy link
Member

@nathanejohnson nathanejohnson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good, I'd like to get another set of eyes on this but it looks like addRoute is only called from NewTable and NewTableCustom , so I don't see this causing issues.

@nathanejohnson
Copy link
Member

So I have to ask... why do you guys have so many routes? :)

@nathanejohnson nathanejohnson added this to the 1.6 milestone Apr 8, 2022
@ddreier
Copy link
Contributor Author

ddreier commented Apr 9, 2022

So I have to ask... why do you guys have so many routes? :)

We like to ask ourselves the same thing 😄

Our Production Nomad clusters each run around 20k Allocations (we run mostly single-tenant services). Depending on the actual application, there could be as many as 6 or 8 routes registered per Alloc. So it ends up that we have Fabio instances for a couple of networks that have 60k routes.

@nathanejohnson nathanejohnson merged commit 8808fc4 into fabiolb:master Apr 11, 2022
@nathanejohnson
Copy link
Member

Thank you for your contribution!

@ddreier ddreier deleted the issue-865-consul-slow-updates branch April 11, 2022 17:40
@ddreier
Copy link
Contributor Author

ddreier commented Apr 11, 2022

Thanks @nathanejohnson! Any hint on when the next version might be released? 😄

@nathanejohnson
Copy link
Member

@ddreier it was just released today.

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.

3 participants