]> git.proxmox.com Git - mirror_ifupdown2.git/commit - ifupdown2/addons/bridge.py
addons: mstpctl: check mstpctl-stp and bridge-stp and fix bridge cache update
authorJulien Fortin <julien@cumulusnetworks.com>
Wed, 18 Mar 2020 03:17:18 +0000 (04:17 +0100)
committerJulien Fortin <julien@cumulusnetworks.com>
Wed, 25 Mar 2020 23:39:03 +0000 (00:39 +0100)
commit24931ffa21e732b0d3a343c8c433702cf0e73e98
treef07f8775b0ea8937e48796a61d09fb3547d0b072
parent8598c690e3c337adda8c00137709f3177a1698e9
addons: mstpctl: check mstpctl-stp and bridge-stp and fix bridge cache update

When an stp is enabled on an existing bridge mstpctl attributes are not always
configured by ifreload. This is due to a timing issue (cache) and some issue in
the mstpctl addon.

- Cache: when changing an existing bridge (done via netlink) we wait for the
kernel ack but we don't update our current cache with the new bridge attributes
This is bad because it means that the bridge cache data are stale until we
receive the notification from the kernel.

- Mstp addon: mstpctl-stp was deprecated in favor of bridge-stp, but in some
place, the mstpctl.py code checks for mstpctl-stp but not for bridge-stp. This
commit fixes the area related to this issue but this should be revisited in
a later commit

Ticket: CM-28951
Reviewed By: Roopa
Testing Done: precommit, smoke, evpn-smoke

Signed-off-by: Julien Fortin <julien@cumulusnetworks.com>
ifupdown2/addons/bridge.py
ifupdown2/addons/mstpctl.py
ifupdown2/lib/nlcache.py