-
Notifications
You must be signed in to change notification settings - Fork 407
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
[feature request]Bookmark and error response should be skipped in yurthub filter. #1868
Labels
Comments
rambohe-ch
added
help wanted
help wanted
kind/good-first-issue
kind/good-first-issue
kind/feature
kind/feature
labels
Dec 22, 2023
/assign |
crazytaxii
added a commit
to crazytaxii/openyurt
that referenced
this issue
Dec 26, 2023
crazytaxii
added a commit
to crazytaxii/openyurt
that referenced
this issue
Dec 26, 2023
rambohe-ch
pushed a commit
that referenced
this issue
Dec 27, 2023
…#1868) (#1873) Signed-off-by: HF <[email protected]>
github-actions bot
pushed a commit
that referenced
this issue
Feb 20, 2024
…#1868) Signed-off-by: HF <[email protected]> (cherry picked from commit 286a43e)
rambohe-ch
pushed a commit
that referenced
this issue
Feb 20, 2024
…#1868) (#1953) Signed-off-by: HF <[email protected]> (cherry picked from commit 286a43e) Co-authored-by: HF <[email protected]>
zyjhtangtang
pushed a commit
to zyjhtangtang/openyurt
that referenced
this issue
Apr 16, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
What would you like to be added:
A built-in filter framework is included in yurthub component, and the response from kube-apiserver can go through these filters.
And response for
watch request
has five types as following:then for
Bookmak
andError
response is not needed to go through filters. so we can add aif condition
to verify thatfilter
should be executed or not as following:openyurt/pkg/yurthub/filter/filter.go
Lines 241 to 244 in 74b0a47
others
/kind feature
The text was updated successfully, but these errors were encountered: