-
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 "syn-only" option to flows-to-world metric #21571
Conversation
b4fc67c
to
aabb253
Compare
/test |
aabb253
to
e9dc28c
Compare
/test |
Use this option to exclude non-SYN drops that could occur transiently from the flows-to-world metric. Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
Include option flags in Status() output so that these flags show up properly in the log message. Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
e9dc28c
to
f97a287
Compare
/test Job 'Cilium-PR-K8s-1.25-kernel-net-next' failed: Click to show.Test Name
Failure Output
If it is a flake and a GitHub issue doesn't already exist to track it, comment |
|
/test-1.25-net-next |
not sure why sig-policy team was added to review, this PR only touches hubble metrics. i'm marking it as ready-to-merge. edit: i guess it's because it modifies a file under |
2 commits:
syn-only
option.Status()
method so that it prints option flags. for example:level=info msg="Configured metrics plugin" name=flows-to-world status="syn-only,destination=dns|reserved-identity,source=namespace" subsys=hubble