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 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
