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

Remove waiting in ipaddress chain element #679

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

NikitaSkrynnik
Copy link
Contributor

Description

This PR fixes the issue that sometimes freezes forwarder-vpp. This is the line that can freeze: line.

@denis-tingaikin
Copy link
Member

@NikitaSkrynnik Could you please add more information about netlink libary bug?

@NikitaSkrynnik
Copy link
Contributor Author

Sure, this PR vishvananda/netlink#793 fixes a goroutine leak which is a root cause of our bug. It's included in netlink v1.3.0, but introduces another bug which is fixed in this PR vishvananda/netlink#1012. It's not incuded in v1.3.0. So we need to wait for another release that includes the second PR.

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

Successfully merging this pull request may close these issues.

2 participants