-
Notifications
You must be signed in to change notification settings - Fork 394
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 command brctl #2971
base: main
Are you sure you want to change the base?
Add command brctl #2971
Conversation
nice! This is going to be very useful. |
4a4d9bf
to
b30fce5
Compare
qemu.ArbitraryArgs("-nic", fmt.Sprintf("user,id=%s", BRCTL_TEST_IFACE_0)), | ||
qemu.ArbitraryArgs("-nic", fmt.Sprintf("user,id=%s", BRCTL_TEST_IFACE_1)), | ||
)), | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we have steps to validate traffic are transferred properly ?
Edit: NVM -- discussed offline, I mis-understood the github UI. I was told this PR was opened from a fork of u-root. @leongross Hi Leon, looks like this Draft is pushed against upstream u-root
Could you push against a fork of u-root, and make this as a PR ? This looks like review-worthy. |
@10000TB I'm not quite sure what kind of review process you suggest. |
you good. I mis-understood the github UI ;) |
2a9acf3
to
032a98a
Compare
3d8137e
to
3772a48
Compare
Signed-off-by: leongross <leon.gross@9elements.com>
Signed-off-by: leongross <leon.gross@9elements.com>
Signed-off-by: leongross <leon.gross@9elements.com>
Signed-off-by: leongross <leon.gross@9elements.com>
…lCase Signed-off-by: leongross <leon.gross@9elements.com>
Signed-off-by: leongross <leon.gross@9elements.com>
Signed-off-by: leongross <leon.gross@9elements.com>
Signed-off-by: leongross <leon.gross@9elements.com>
Signed-off-by: leongross <leon.gross@9elements.com>
conversion Signed-off-by: leongross <leon.gross@9elements.com>
Signed-off-by: Christopher Meis <christopher.meis@9elements.com>
Signed-off-by: leongross <leon.gross@9elements.com>
Signed-off-by: leongross <leon.gross@9elements.com>
Signed-off-by: leongross <leon.gross@9elements.com>
Signed-off-by: leongross <leon.gross@9elements.com>
Signed-off-by: Christopher Meis <christopher.meis@9elements.com>
7a4b249
to
67d8c32
Compare
Signed-off-by: Christopher Meis <christopher.meis@9elements.com>
Well, tests run or being skipped if eth0/eth1 are not present in setup. Dunno how to fix the coverage complains. |
If you look at each file in this PR, there are alerts from "Codecov / codecov/patch" at each specific line telling you relevant lines are missing tests. The codecov seems to suggest this PR brings down coverage by
|
Add
cmd/brctl
which aims to be a (near) feature complete implementation of brctl.TODOs