On Thu, Apr 14, 2022 at 4:06 AM Dumitru Ceara <[email protected]> wrote: > > On 4/1/22 12:16, Xavier Simonart wrote: > > Before this patch, when all flows for a port have been installed in OVS, > > ovn-installed was immediately written to the OVS DB; the notification of > > this change to the OVN controller (handled by OVS interface handler) > > might cause a full recompute if the SBDB is readonly / still handling > > the chassis update in PortBinding. > > With this patch, ovn-installed is not written to the OVS DB while the > > SBDB is readonly. This reduces the number of (un-necessary) full > > recomputes. > > Note that this was causing some spurious full recompute in test "ACL > > with Port Group conjunction flow efficiency", causing the test to fail > > from time to time. The test case was also updated as part of this patch > > to check that ovn-installed was always properly set. > > > > Signed-off-by: Xavier Simonart <[email protected]> > > > > --- > > It seems that delaying the "ovn-installed" causes measurable latency > spikes when scale testing with OpenShift/ovn-kubernetes. On a 120 nodes > cluster we saw "ovn-installed" being delayed occasionally for up to > 500ms. All this time the OVS flows were already properly installed but > the SB transaction was still in progress. > > We might have to figure out a different way to address the issue this > patch is fixing. > > Thanks, > Dumitru > Hi Dumitru, Xavier, Lorenzo,
I used a different approach to fix the problem. Please take a look: https://patchwork.ozlabs.org/project/ovn/patch/[email protected]/ Thanks, Han _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
