Hide Forgot
Description of problem: The user should be warned when working with bonding & SR-IOV and attempts to bond 2 VFs under the same physical port. Bonding 2 VFs under the same physical port doesn't make sense.
Doesn't network manager warn on this? If not can we move this RFE to them?
Defining a bond over VF/s that share the same PF is not only wrong, it is an error. The phy has a mini switch/bridge and having potential duplicate mac/s arriving from different VF/s will mess it up. The RFE should not be on adding the warning, it should be on hiding the VF interfaces on the host, only VM/s should use those and I do not see any valid scenario to have them exposed on the host. Regarding NM, I am not sure how we can expect such a warning from NM, it is not concerned with high lever misuse of the devices, that's a higher level concern.
(In reply to Edward Haas from comment #2) > Defining a bond over VF/s that share the same PF is not only wrong, it is an > error. The phy has a mini switch/bridge and having potential duplicate mac/s > arriving from different VF/s will mess it up. > > The RFE should not be on adding the warning, it should be on hiding the VF > interfaces on the host, only VM/s should use those and I do not see any > valid scenario to have them exposed on the host. Containers may use these devices. > > Regarding NM, I am not sure how we can expect such a warning from NM, it is > not concerned with high lever misuse of the devices, that's a higher level > concern. I don't think so, since this is the host level error, not a cluster level error.
(In reply to Yaniv Lavi from comment #3) > (In reply to Edward Haas from comment #2) > > Defining a bond over VF/s that share the same PF is not only wrong, it is an > > error. The phy has a mini switch/bridge and having potential duplicate mac/s > > arriving from different VF/s will mess it up. > > > > The RFE should not be on adding the warning, it should be on hiding the VF > > interfaces on the host, only VM/s should use those and I do not see any > > valid scenario to have them exposed on the host. > > Containers may use these devices. Containers in this regard have the same behaviour, the VF interface is moved to the context of the VM/Container. I claim that they should not be used in the context/view of the host. > > > > > Regarding NM, I am not sure how we can expect such a warning from NM, it is > > not concerned with high lever misuse of the devices, that's a higher level > > concern. > > I don't think so, since this is the host level error, not a cluster level > error. It is a user error, the tool allows you to configure things, but it is not enforcing you to do it in a "correct" way. Someone may find such a use valid, like in a test. But I guess you need to convince NM team to see it as you do, not me. Maybe it will make sense for them.
Closing old RFEs, please reopen if still needed. Patches are always welcomed.