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

Downgrade netlink library version #699

Merged
merged 1 commit into from
Jun 25, 2018
Merged

Downgrade netlink library version #699

merged 1 commit into from
Jun 25, 2018

Conversation

jellonek
Copy link
Contributor

@jellonek jellonek commented Jun 20, 2018

While vishvananda/netlink#354 is not resolved we need to downgrade version of netlink library to avoid issue described in #629
Upgrade of this lib was needed by vlans support for sr-iov, but it's now postponed for a while, so there is no reason to keep this lib in version with that issue.


This change is Reviewable

@pigmej
Copy link
Contributor

pigmej commented Jun 22, 2018

:lgtm:


Reviewed 2 of 2 files at r1.
Review status: 0 of 2 LGTMs obtained, and 1 stale


Comments from Reviewable

@ivan4th
Copy link
Contributor

ivan4th commented Jun 25, 2018

:lgtm:


Review status: 1 of 2 LGTMs obtained, and 1 stale


Comments from Reviewable

@ivan4th ivan4th merged commit 1668790 into master Jun 25, 2018
@ivan4th ivan4th deleted the jell/downnetlink branch June 25, 2018 12:16
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.

3 participants