search

Exa-Networks / exabgp

The BGP swiss army knife of networking
Other
2.05k stars 440 forks source link

Parsing some l2vpn-evpn messages causes session flap #1195

Closed ottorei closed 4 weeks ago

ottorei commented 6 months ago

With version 4.2.21 peering with Juniper MX router as a route reflector client with family l2vpn-evpn, the BGP-session seems to constantly flap. Attachment contains the last errors being logged on ExaBGP side.

[exa-errors.txt]

thomas-mangin commented 6 months ago

Please provide enough information for me to help. Output of the full log using exabgp -d.

ottorei commented 6 months ago

The full log file would be quite large since there are over 100k evpn routes being received. Would it help to capture the single raw message that causes this issue? Also, the full log file would expose sensitive information about the production network

thomas-mangin commented 6 months ago

When using -d the logs contain the exchanged open messages and shared updates. The lines with the opens and way down the update before the crash is what I would need. A few other working similar updates would also be good to add to the test suite. Thank you

thomas-mangin commented 6 months ago

Also please check with the main branch as this is where I will fix it.

ottorei commented 6 months ago

When using -d the logs contain the exchanged open messages and shared updates. The lines with the opens and way down the update before the crash is what I would need. A few other working similar updates would also be good to add to the test suite. Thank you

Thanks. Perhaps this log has this information. At least there is one working decode before the crash. [exa-mpls-evpn-debug.txt]

ottorei commented 6 months ago

Any change getting this fix merged on 4.2 release?

thomas-mangin commented 5 months ago

@ottorei I could backport but no one confirmed that the change worked as expected.

ottorei commented 5 months ago

@ottorei I could backport but no one confirmed that the change worked as expected.

I'll patch my local instance and try it later this week.

thomas-mangin commented 4 weeks ago

no news good news. closing as fixed in master.