We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Would like to dynamically remove event handlers from informers. Similarly, how it is implemented in go: https://github.com/kubernetes/client-go/blob/3dc7fd5f4c1d8afaf5924c461eae2ab27db0045a/tools/cache/shared_informer.go#L166
Similarly as in go, would like to change the return type of the function:
https://github.com/fabric8io/kubernetes-client/blob/refs/heads/main/kubernetes-client/src/main/java/io/fabric8/kubernetes/client/informers/impl/DefaultSharedIndexInformer.java#L100
to a new type ResourceEventHandlerRegistration. That could be subsequently used to remove the event handler.
ResourceEventHandlerRegistration
No response
The text was updated successfully, but these errors were encountered:
@shawkins @manusa if this makes sense to you, I can take a look and implement it.
Sorry, something went wrong.
if this makes sense to you, I can take a look and implement it.
Removing handlers seems absolutely fine.
OK from my side too. You're the ones mostly dealing with informers. Please go ahead 🙌
csviri
No branches or pull requests
Is your enhancement related to a problem? Please describe
Would like to dynamically remove event handlers from informers.
Similarly, how it is implemented in go:
https://github.com/kubernetes/client-go/blob/3dc7fd5f4c1d8afaf5924c461eae2ab27db0045a/tools/cache/shared_informer.go#L166
Describe the solution you'd like
Similarly as in go, would like to change the return type of the function:
https://github.com/fabric8io/kubernetes-client/blob/refs/heads/main/kubernetes-client/src/main/java/io/fabric8/kubernetes/client/informers/impl/DefaultSharedIndexInformer.java#L100
to a new type
ResourceEventHandlerRegistration
. That could be subsequently used to remove the event handler.Describe alternatives you've considered
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: