-
Notifications
You must be signed in to change notification settings - Fork 331
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
FilterFunc in the informer is not working on the first run of reconciler #1764
Comments
Yup, there's no filter passed to genreconciler currently, but you could plumb one through in the controller options. |
@mattmoor can you point to an example of plumbing through? Looks like the code directly enqueues every object in the lister, skipping any informer FilterFunc. |
By "plumbing through" I mean adding a field and calling it from that logic in genreconciler itself, there is no field today. |
This issue is stale because it has been open for 90 days with no |
/lifecycle frozen |
/kind bug
Expected Behavior
The informer should always successfully filter objects based on filterFunc defined the informer
Actual Behavior
In the first run of the reconcile, it lists all objects and objects are not filtered by filterFunc in the informer
Steps to Reproduce the Problem
Additional Info
We did some investigation, in the genreconciler: https://github.com/google/knative-gcp/blob/3707bd3ca3d010532b334201dc7da6c9bc382959/pkg/client/injection/reconciler/broker/v1beta1/broker/controller.go#L73-L81
When a controller starts, a new leader is promoted with a listAll operation and there is no informer resync.
The text was updated successfully, but these errors were encountered: