-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Hubble: add option to filter for pods and services in any namespace #28921
Conversation
3f68127
to
fa56001
Compare
fa56001
to
71d7fce
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Interesting idea, this seems reasonable to me.
/test |
71d7fce
to
c01f7e3
Compare
@rolinh PTAL. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! lgtm
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
/test |
We now allow service and pod filters in the form of `/foo`, which will filter for any service/pod starting with `foo` in any namespace. Signed-off-by: Fabian Fischer <fabian.fischer@isovalent.com>
c01f7e3
to
a5cb9c6
Compare
The CI failures seem unrelated and most likely was because this branch was >400 commits behind main/CI. Rebased on main. Could you please re-trigger tests? :) |
/test |
/ci-ipsec-upgrade |
/ci-ginkgo |
/ci-e2e |
/ci-clustermesh |
Please ensure your pull request adheres to the following guidelines:
description and a
Fixes: #XXX
line if the commit addresses a particularGitHub issue.
We now allow service and pod filters in the form of
/foo
, which will filter for any service/pod starting withfoo
in any namespace.Related to: cilium/hubble#359