-
Notifications
You must be signed in to change notification settings - Fork 414
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
Unregister unnecessary configMap informer #1273
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: savitaashture The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Nice find @guillaumerose We used to create one logging configmap per namespace for EL logging config but in #1207 we switched to passing this via env vars /lgtm |
7700a03
to
8903df2
Compare
I rebased the PR and also removed configMaps from the controller test. |
The following is the coverage report on the affected files.
|
It prevents controller to watch for configMap in all namespaces. Before this change: controller logs shows: 2021/11/25 08:46:36 Registering 5 informers crictl stats reports 66.88MB with around 4k configMaps of 2.5kB with kind. After this change: controller logs shows: 2021/11/25 08:48:48 Registering 4 informers crictl stats reports 16.29MB Related to tektoncd#1268
8903df2
to
41eb213
Compare
The following is the coverage report on the affected files.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
/lgtm
Changes
It prevents controller to watch for configMap in all namespaces.
Before this change:
controller logs shows:
2021/11/25 08:46:36 Registering 5 informers
crictl stats reports 66.88MB with around 4k configMaps of 2.5kB with kind.
After this change:
controller logs shows:
2021/11/25 08:48:48 Registering 4 informers
crictl stats reports 16.29MB
Related to #1268
Submitter Checklist
These are the criteria that every PR should meet, please check them off as you
review them:
See the contribution guide for more details.
Release Notes