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 the ability to scale the application-set controller more. #16945

Closed
tomelliot16 opened this issue Jan 22, 2024 · 3 comments · Fixed by #18623
Closed

Add the ability to scale the application-set controller more. #16945

tomelliot16 opened this issue Jan 22, 2024 · 3 comments · Fixed by #18623
Labels
component:application-sets Bulk application management related enhancement New feature or request

Comments

@tomelliot16
Copy link

Summary

  • Have the ability to scale up max reconciles larger then 100
  • Have the ability to raise qps and burst for kubernetes api.

Motivation

  • We are running into scale issues

Proposal

  • Raise the maxReconciles max to 500 in the application-set-controller bin arg
  • Add qps and burst as args to the application-set-controller bin
@tomelliot16 tomelliot16 added the enhancement New feature or request label Jan 22, 2024
@blakepettersson
Copy link
Member

I know that @JorTurFer was interested in working on this, pending first profiling the Appset controller to see where the actual bottlenecks are (see #16594).

@JorTurFer
Copy link
Contributor

Yeah, I've it on my TODOs list, but currently there are too many things on my plate... 😢

@keithchong keithchong added the component:application-sets Bulk application management related label Jan 22, 2024
@alexymantha
Copy link
Member

I'm also running into this, I need to increase QPS and Burst. I will try to get a PR in this week

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:application-sets Bulk application management related enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants