Bug 2116119 - Add support for vdsm to use nmstate for bridge options
Summary: Add support for vdsm to use nmstate for bridge options
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: vdsm
Classification: oVirt
Component: Core
Version: 4.50.2.2
Hardware: x86_64
OS: Linux
medium
medium
Target Milestone: ovirt-4.5.3
: ---
Assignee: eraviv
QA Contact: Michael Burman
URL:
Whiteboard:
Depends On: 2117485 2117704
Blocks:
TreeView+ depends on / blocked
 
Reported: 2022-08-07 08:52 UTC by Michael Burman
Modified: 2022-09-20 09:21 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2022-09-20 09:21:23 UTC
oVirt Team: Network
Embargoed:
mperina: ovirt-4.5+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github oVirt vdsm pull 283 0 None Draft Network: support nmstate bridge STP options 2022-08-08 06:59:24 UTC
Red Hat Issue Tracker RHV-47807 0 None None None 2022-08-07 08:54:56 UTC

Description Michael Burman 2022-08-07 08:52:19 UTC
Description of problem:
Add support for vdsm to use nmstate for bridge options

In BZ 2108974 we have merged a vdsm code to use nmstate for bridge_opts options in RHV. While the code merged we have discovered that some parameters are broken, as nmstate moved(many versions ago) some of them under stp, instead of the common options. Like 'priority' and more. 
This bug will track the full support of vdsm to use nmstate for bridge_opts in RHV.

Version-Release number of selected component (if applicable):
vdsm-4.50.2.2-1.el8ev.x86_64

Comment 1 eraviv 2022-08-07 10:25:35 UTC
stp options reported by nmstate [1] :

      stp:
        enabled: false
        forward-delay: 15
        hello-time: 2
        max-age: 20
        priority: 32768

https://github.com/nmstate/nmstate/blob/nmstate-1.2/libnmstate/schema.py#L282

Comment 2 eraviv 2022-09-20 09:21:23 UTC
In el8 not only nmstate ignores bridge options when STP is disabled but also NetworkManager.
That's because when NM is configured to use /etc/sysconfig/network-scripts/ifcfg files it ignores STP options when STP is disabled.
In el8 ifcfg files are the default for NetworkManager.

Therefore we decide to close this bug as WONTFIX, because changing the default NetworkManager behavior is out of scope for oVirt\RHV at this point in time.


Note You need to log in before you can comment on or make changes to this bug.