-
Notifications
You must be signed in to change notification settings - Fork 24
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
sFlow Support #10
Comments
Depends on if you can find a sFlow decoder library for Go? (I haven't honestly looked to see.) Also, sadly I have no devices which generate sFlow so I have no means of testing the code. So that's a major blocker right now. |
Hello synfinatic, Doesn't goflow support sflow? https://github.com/cloudflare/goflow, I'd be happy to help with testing. |
oh, so it does. you've got better eyes than i do. Honestly, I have no way of testing sFlow and that sucks. If you're up for submitting a patch I'd be willing to look at it, but without any means of doing my own testing during development (not to mention having no need for the feature) means it's unlikely to happen anytime soon. |
How difficult would it be to add sFlow support? I was looking to implement this as a drop in replacement for nProbe and its ridiculous fee, but our Brocade uses sFlow instead of netflow.
The text was updated successfully, but these errors were encountered: