-
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 a flag to write Hubble events to a rotated file #15557
Conversation
fixing test failures... |
test-me-please |
test-gke |
test-1.16-netnext |
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 work! One small comment
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 work! Some non-blocking suggestions below.
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.
🚀 LGTM apart from one small nit and what Sebastian and Robin already pointed out.
thank you all for your review. addressed all the comments except for #15557 (comment) will add more test cases tomorrow 👍 |
added some more test cases |
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.
LGTM 🚀
test-me-please |
Add a flag to write Hubble events from OnDecodeEvent() to a rotated file. It could be useful for troubleshooting to have access to recent events beyond what's kept in memory. Use it with caution as it can add significant overhead on busy nodes. Signed-off-by: Michi Mutsuzaki <michi@isovalent.com>
test-me-please |
I checked that all team review requests are covered. Tests are passing. Merging. |
Add a flag to write Hubble events from OnDecodeEvent() to a rotated
file. It could be useful for troubleshooting to have access to recent
events beyond what's kept in memory. Use it with caution as it can
add significant overhead on busy nodes.
Signed-off-by: Michi Mutsuzaki michi@isovalent.com