Skip to content

Allow configuration of the number of informer workers #140

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

Closed
mikewied opened this issue Mar 28, 2018 · 2 comments
Closed

Allow configuration of the number of informer workers #140

mikewied opened this issue Mar 28, 2018 · 2 comments
Assignees
Labels
kind/feature Categorizes issue or PR as related to a new feature. needs discussion

Comments

@mikewied
Copy link

Currently the number of workers in the informer is hardcoded to 1. Being able to set this number higher will allow operator to process multiple CRD updates at once. This will help the operator be more responsive when it is managing a large number of CRD's.

Note: I think 1 is definitely the right default.

@hasbro17 hasbro17 added kind/feature Categorizes issue or PR as related to a new feature. needs discussion labels Jun 5, 2018
@ib-steffen
Copy link

ib-steffen commented Jun 13, 2018

is there any reason why it has to be 1 at the moment? It seems that our operator is currently limited by this, because we manage many CRs at the same time. In our case we also create/delete them all the time, so there's really quite some work to be done by the operator.

@theishshah
Copy link
Member

Number of Workers is now configurable!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature. needs discussion
Projects
None yet
Development

No branches or pull requests

4 participants