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.

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

sql: silence informer errors on unwatchable resources #284

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

moio
Copy link
Contributor

@moio moio commented Sep 24, 2024

Listable-but-unwatchable resources such as nodes metrics.k8s.io/v1beta1 allow an Informer to be started, but it will by default log all watch failures.

Detect unwatchable types and tell Lasso so that errors can be silenced

partially addresses rancher/rancher#46030

best reviewed commit-by-commit

moio added 3 commits December 3, 2024 09:45
Signed-off-by: Silvio Moioli <silvio@moioli.net>
Signed-off-by: Silvio Moioli <silvio@moioli.net>
Signed-off-by: Silvio Moioli <silvio@moioli.net>
@moio moio force-pushed the detect_unwatchable_listable branch from bfea089 to d3405f3 Compare December 3, 2024 08:45
moio added 2 commits December 3, 2024 09:56
Signed-off-by: Silvio Moioli <silvio@moioli.net>
Signed-off-by: Silvio Moioli <silvio@moioli.net>
@moio moio force-pushed the detect_unwatchable_listable branch from d3405f3 to bdb70ad Compare December 3, 2024 08:59
@prachidamle prachidamle removed the request for review from a team December 11, 2024 21:12
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.

1 participant