Hi guys, Could you please review these patches?
Thank you, Alexander On 3/18/25 1:08 PM, Aleksandr Smirnov wrote: > First commit delays lflow uploads to OVS until all monitored updates come > to the controller. This is done by monitor condition sequence number > comparison vs expected sequence number. A lflow update is released when > current number becomes equal to the expected number. Theoretically this should > work for any number of monitored updates. > Practically, for today, there are exactly 3 flow updates if monitor-all=false > and 1 update if monitor-all=true. > > You can still use the wait-before-clear != 0 option if needed, this will > disable automated delay. > > Since wait-before-clear seems to be fully replaced with the first commit > you may want to remove wait-before-clear functionality and this is done with > the second commit. > > Aleksandr Smirnov (2): > controller: Delay initial flow upload to OVS. > controller: Remove wait-before-clear option. > > controller/ofctrl.c | 45 +++++++--------- > controller/ofctrl.h | 3 +- > controller/ovn-controller.c | 4 +- > tests/ovn-controller.at | 104 +++++++++++------------------------- > 4 files changed, 55 insertions(+), 101 deletions(-) > _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
