-
Notifications
You must be signed in to change notification settings - Fork 5
Pr/daniel noland/host gateway #622
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
Closed
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
e354bfd
to
19601f5
Compare
These have a few unused dependencies, unsorted lists, and stray feature activations. Signed-off-by: Daniel Noland <[email protected]>
Signed-off-by: Daniel Noland <[email protected]>
This makes logs easier to read. Signed-off-by: Daniel Noland <[email protected]>
This makes logs easier to read. Signed-off-by: Daniel Noland <[email protected]>
This makes logs easier to read. Signed-off-by: Daniel Noland <[email protected]>
This makes logs easier to read. Signed-off-by: Daniel Noland <[email protected]>
This makes logs easier to read. Signed-off-by: Daniel Noland <[email protected]>
This quiets lints about symbols in backticks in the docs. Signed-off-by: Daniel Noland <[email protected]>
Various style issues triggering the linter corrected. The only material change is the renae of a macro to fit the standard rust macro casing convention. Signed-off-by: Daniel Noland <[email protected]>
Linux frequently reserves zero in netlink and that includes interface index. Better to check than to issue nonsense commands to linux. Signed-off-by: Daniel Noland <[email protected]>
There is enough logic here that the code is easier to organize with Mtu in another file. Signed-off-by: Daniel Noland <[email protected]>
This adds incomplete support for reading PCI netdevs from netlink. The missing feature is mapping them back to a specific PCI EBDF address (e.g. 0000:01:00.0). I will add support for that in a follow-on commit (where we need to switch to a Hedgehog fork of rtnetlink). Signed-off-by: Daniel Noland <[email protected]>
This is needed to extend bridge in the next commit. Signed-off-by: Daniel Noland <[email protected]>
This is needed to get information about the bridge fdb out of linux. Required for host vtep support. Signed-off-by: Daniel Noland <[email protected]>
Signed-off-by: Daniel Noland <[email protected]>
19601f5
to
7a7a796
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
scratch, don't review yet