-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add: Added support for ebpf tcptop - Issue#6 #7
Add: Added support for ebpf tcptop - Issue#6 #7
Conversation
032d475
to
97c8e07
Compare
97c8e07
to
116743d
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.
Thanks for the the review William :)
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.
@maitreya-source Thanks a lot for the PR 💯
Other than that last comment, the changes LGTM
I will merge it once you're ready 🚀
Btw, do you have a sample ebpf_exporter config that we can include in README.md later?
116743d
to
8afc0ed
Compare
Ready to merge :) For ebpf_exporter config, I have a yaml file ready to be put as reference:
|
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.
🚀
Issue #6