Skip to content
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 netlink Statistics to nodes #109

Closed
safchain opened this issue Sep 6, 2016 · 8 comments
Closed

Add netlink Statistics to nodes #109

safchain opened this issue Sep 6, 2016 · 8 comments
Assignees

Comments

@safchain
Copy link
Collaborator

safchain commented Sep 6, 2016

Add Statistics returned by netlink to interface nodes so that we can get dropped packet number, etc...
This need to add an async mechanism to get metrics in order to have them updated.

@matrohon
Copy link
Contributor

matrohon commented Dec 8, 2016

I see two possible go libraries to do that :

@safchain
Copy link
Collaborator Author

safchain commented Dec 9, 2016

Something we have to take care is the way the library is implemented especially around the syscalls as we got issues with namespace and goroutine. Something fixed in ethtool lib but that doesn't mean this is the best choice.

@matrohon
Copy link
Contributor

matrohon commented Dec 9, 2016

I'm currently playing with the ethtool lib, and indeed, I'm having issues with syscall. I was suggesting your ethtool lib to avoid another dependency. but it might be safer to use gopsutil that only poll net/dev files. BTW, should it be treated as a new probe?

@safchain
Copy link
Collaborator Author

safchain commented Dec 9, 2016

I meant that ethtool is safe with namespaces, not sure about gopsutil. And yes it should be treated as another probe polling interface metrics. I'll have a look at gopsutil to be sure.

@safchain
Copy link
Collaborator Author

safchain commented Dec 9, 2016

I had at quick look at gopsutil and indeed no namespace safe, so If we want to use it we will have to send some PR to fix it.

@safchain
Copy link
Collaborator Author

safchain commented Dec 9, 2016

What kind of issue you have with ethtool ? maybe you could open an issue

@matrohon
Copy link
Contributor

matrohon commented Dec 9, 2016

done :
safchain/ethtool#11

@lebauce lebauce self-assigned this Jan 23, 2017
@safchain
Copy link
Collaborator Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants