On Mon, May 5, 2025 at 5:33 AM Smirnov Aleksandr (K2 Cloud) <AleksSmirnov@k2.cloud> wrote: > > Hi guys, > > Could you please review these patches?
Sorry for this. I'll take a look. Numan > > 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 _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
