Skip to content

[release-v1.16] Reduce mt-broker-controller memory usage with namespaced endpoint informer (#8418)#1012

Merged
pierDipi merged 1 commit intoopenshift-knative:release-v1.16from
openshift-cherrypick-robot:cherry-pick-1002-to-release-v1.16
Jan 29, 2025
Merged

[release-v1.16] Reduce mt-broker-controller memory usage with namespaced endpoint informer (#8418)#1012
pierDipi merged 1 commit intoopenshift-knative:release-v1.16from
openshift-cherrypick-robot:cherry-pick-1002-to-release-v1.16

Conversation

@openshift-cherrypick-robot

This is an automated cherry-pick of #1002

/assign pierDipi

…ormer (knative#8418)

* Reduce mt-broker-controller memory usage with namespaced endpoint informer

Currently, the mt-broker-controller is using a cluster-wide endpoints
informer but it actually only uses endpoints in the "SYSTEM_NAMESPACE".

Using the namespaced informer factory ensures that the watcher
is only watching endpoints in the `knative-eventing` (also known as
`SYSTEM_NAMESPACE`) namespace.

Signed-off-by: Pierangelo Di Pilato <pierdipi@redhat.com>

* Start informer

Signed-off-by: Pierangelo Di Pilato <pierdipi@redhat.com>

---------

Signed-off-by: Pierangelo Di Pilato <pierdipi@redhat.com>
@openshift-ci
Copy link

openshift-ci bot commented Jan 28, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: openshift-cherrypick-robot
Once this PR has been reviewed and has the lgtm label, please assign skonto for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@pierDipi pierDipi merged commit 125b0dc into openshift-knative:release-v1.16 Jan 29, 2025
5 of 58 checks passed
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.

2 participants