-
Notifications
You must be signed in to change notification settings - Fork 288
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
create bridge on-demand #84
Comments
This complicates a few things like deleting the bridge or not on destroy. So a few decisions need be made when implementing this:
|
The bridge can be deleted only if it has no interfaces attached to it, so I
think its ok to keep the flow as follows:
1. create the bridge if its not created already upon the lab deployment
2. delete the bridge on destroy if no interfaces are connected (interfaces
got deleted first)
…On Wed, Oct 7, 2020 at 12:04 PM Karim Radhouani ***@***.***> wrote:
This complicates a few things like deleting the bridge or not on destroy.
The bridge idea is meant to be used to connect different fabrics together,
so it can belong to multiple fabrics.
So a few decisions need be made when implementing this:
- delete the bridge or just the iterface on destroy, can be controlled
via a flag
- add a few config options under a bridge dut like mtu or rely on
default values... Might be incompatible when the bridge is used by multiple
fabrics.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#84 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABLKV5NNN4FDF5OJ6XBG4XDSJQ4MLANCNFSM4SHFGRUA>
.
|
In this case we need to make sure that changes like mtu and offloading are not applied to already existing bridges |
Dup of #2267 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
When link' kind is set to
bridge
and the bridge does not exist - create the bridge automatically with the tcp offload set tooff
and default MTU set to9212
The text was updated successfully, but these errors were encountered: