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.

Sign up for GitHub

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

Jump to bottom

allow filtering watcher events when !observedGeneration changed #52

Closed
clux opened this issue Jul 20, 2019 · 2 comments · Fixed by #911
Closed

allow filtering watcher events when !observedGeneration changed #52

clux opened this issue Jul 20, 2019 · 2 comments · Fixed by #911
Assignees
@clux
Labels
runtime controller runtime related

Comments

@clux
Copy link
Member

clux commented Jul 20, 2019 •
edited
Loading

rationale: we don't necessarily want to receive events when the .status object changes
@clux clux changed the title allow limiting informer events to when observed generation changed allow limiting informer events to when observedGeneration changed Jul 20, 2019
@clux clux removed the enhancement label Feb 27, 2020
@clux clux added runtime controller runtime related good first issue Good for newcomers help wanted Not immediately prioritised, please help! labels Mar 8, 2020
@clux
Copy link
Member Author

clux commented Mar 8, 2020 •
edited
Loading

This would require maintaining a map of ObjectRef -> observedGeneration (int) inside a watcher / reflector and checking if it changed before bubbling up the event (first added event would always succeed)

Not quite what the best way to extend this, it's not very nice to have this in the internal watcher state by default, so need to have this optional somehow. It's a bit of a niche feature, and honestly, mostly for controllers to filter out its own modifications, so probably should think about this holistically in #279

@clux clux mentioned this issue Mar 10, 2020
Closed
@clux clux removed good first issue Good for newcomers help wanted Not immediately prioritised, please help! labels Aug 10, 2020
@clux clux changed the title allow limiting informer events to when observedGeneration changed allow filtering watcher events when !observedGeneration changed Nov 3, 2021
clux added a commit to kube-rs/controller-rs that referenced this issue May 13, 2022
@clux
PoC Predicates for kube-rs/kube#52
7e2a152 
Signed-off-by: clux <sszynrae@gmail.com>
@clux clux self-assigned this May 13, 2022
@clux
Copy link
Member Author

clux commented May 13, 2022

PoC in kube-rs/controller-rs@7e2a152
using the impl trait magic that @teozkr brought for runtime conditions.
I think this is viable in terms of inclusion, but there's probably some cleanups. Feel free to drop some comments, otherwise I'll prepare a PR for that and await reviews then.

@clux clux linked a pull request May 13, 2022 that will close this issue
Add predicates to allow filtering watcher streams #911
Merged
clux added a commit to kube-rs/controller-rs that referenced this issue May 23, 2022
@clux
0.73.0 sync (#29)
2976e04 
* Use kube-rs/kube#910

Signed-off-by: clux <sszynrae@gmail.com>

* PoC Predicates for kube-rs/kube#52

Signed-off-by: clux <sszynrae@gmail.com>

* remove predicate stuff

Signed-off-by: clux <sszynrae@gmail.com>

* undo crd update

Signed-off-by: clux <sszynrae@gmail.com>

* build from branch

Signed-off-by: clux <sszynrae@gmail.com>

* kube bump

Signed-off-by: clux <sszynrae@gmail.com>

* bump

Signed-off-by: clux <sszynrae@gmail.com>
@clux clux closed this as completed in #911 Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@clux clux

Labels
runtime controller runtime related
Projects
Kube Roadmap
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add predicates to allow filtering watcher streams kube-rs/kube
1 participant
@clux