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 support of ExtendedMPLS for sflow #327

Open
akarneliuk opened this issue May 22, 2024 · 1 comment
Open

Add support of ExtendedMPLS for sflow #327

akarneliuk opened this issue May 22, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@akarneliuk
Copy link

Is your feature request related to a problem? Please describe.
the sflow v5 has implemented extended mpls struct, which contains mpls data. This is not implemented in goflow2 sflow. As such, i cannot properly decode mpls labels

Describe the solution you'd like
To add support of mpls extended mpls to sflow part of goflow2 to make this information available upon parsing.

Describe alternatives you've considered
There is not much alternative here. I am. Using telegraf, which relies in this library for sflow parsing.

Additional context
https://sflow.org/SFLOW-STRUCTS5.txt

@akarneliuk akarneliuk added the enhancement New feature or request label May 22, 2024
@lspgn
Copy link
Member

lspgn commented May 23, 2024

Thank you @akarneliuk for the suggestion. This should be possible. GoFlow2 already decodes MPLS so all the fields already exist in the protobuf.

Although: what is the model of your device producing sFlow? Have you tried with regular Ethernet samples?
Do you have a packet capture?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants