search

bisdn / basebox

A tiny OpenFlow controller for OF-DPA switches.
Mozilla Public License 2.0
45 stars 9 forks source link

VxLAN stability fixes #409

Closed KanjiMonster closed 1 year ago

KanjiMonster commented 1 year ago

Improve VxLAN setup and handle various orders of operations (again) that got broken with https://github.com/bisdn/basebox/commit/935d0e6fbd9a958a32189638be78ff5fbc7a2cc4.

Description

Fix VxLAN setup handling by properly notifying the bridge about VxLAN attachement again, and don't try to call vxlan::create_endpoint() twice for late attachment / VxLAN creation.

Motivation and Context

Fixes various VxLAN setup failures, and potentially random instability in tests.

How Has This Been Tested?

Ran the testing pipeline on as4630-54pe, with all tests green.

KanjiMonster commented 1 year ago

For some reason these fixes don't work as good as expected, will need to investigate.

KanjiMonster commented 1 year ago

Fixed missing case of attaching a newly created vxlan after bridge take over.

rubensfig commented 1 year ago

LGTM