-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
feat: socketstat input plugin #3649
Conversation
… tests, unix sockets support.
@sajoupa Thanks for the pull request. I think there may be two high level concerns with this pr:
|
Hi @danielnelson , you're totally right about data cardinality, I added a warning to the README. I have also updated the code to make the same changes which had been requested for the ipset input plugin (use a timeout, buffio scanner, ... ref #3346) |
I'm not sure if I will want to merge this in, in the meantime we can leave the pr open for anyone who would like to use it. |
@danielnelson which of the two reasons you first mentioned would block this ? |
Shelling out to ss is the bigger issue, we can add warnings to the documentation about cardinality and how to deal with it. |
@danielnelson Hi, I looked further into it and think shelling out is the only option. |
Co-authored-by: Sven Rebhan <36194019+srebhan@users.noreply.github.com>
Co-authored-by: Sven Rebhan <36194019+srebhan@users.noreply.github.com>
Co-authored-by: Sven Rebhan <36194019+srebhan@users.noreply.github.com>
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 your patience @sajoupa! There are a few additional improvements I'd like to see going in. Some are just a matter of taste, so please speak-up if you don't like them.
Co-authored-by: Sven Rebhan <36194019+srebhan@users.noreply.github.com>
…ich always returns nil
Co-authored-by: Sven Rebhan <36194019+srebhan@users.noreply.github.com>
…rror processing them Co-authored-by: Sven Rebhan <36194019+srebhan@users.noreply.github.com>
…ecause in test envs where `ss` might be absent, we still need to perform the other actions
📦 Looks like new artifacts were built from this PR. Expand this list to get them here ! 🐯Artifact URLs |
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.
Awesome! Thanks for driving this home @sajoupa! LGTM.
Cheers, thanks for the patient reviews. |
Required for all PRs:
Hi,
The
ss
command (package iproute2) gives very useful metrics about sockets on Linux machines, this is a plugin to expose them with telegraf.Thanks.