Hi Lorenzo, On 5/13/22 19:03, Lorenzo Bianconi wrote: > ovn-main: > --------- > Statistics for 'lflows_lbs' > Total samples: 51 > Maximum: 1319 msec > Minimum: 1282 msec > 95th percentile: 1309.836278 msec > Short term average: 1296.054252 msec > Long term average: 1295.728676 msec > > ovn-main+optimization: > ---------------------- > Statistics for 'lflows_lbs' > Total samples: 51 > Maximum: 526 msec > Minimum: 503 msec > 95th percentile: 524.794986 msec > Short term average: 518.826313 msec > Long term average: 509.005444 msec
This is a nice improvement! I had a look at the changes and they seem OK to me. I acked the patches but maybe others can have another look too before merging these. > > Changes since v1: > - group together distributed routers similar to gw router > - fix force_snat flag bug > > Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2079834 Maybe the maintainers can do this when applying the series, otherwise can you please add the "Reported-at" tag to the individual patches? Like that our automation will pick up the reference between the OVN commits and the BZ. > > Lorenzo Bianconi (5): > northd: move nat_flows_for_lb for skip_snat in a dedicated routine > northd: move nat_flows_for_lb for force_nat in a dedicated routine > northd: move nat_flows_for_lb for gw router in a dedicated routine > northd: refactor logical router loop in build_lrouter_nat_flows_for_lb > northd: optimize build_gw_lrouter_nat_flows_for_lb code > > northd/northd.c | 170 ++++++++++++++++++++++++++++++++++++------------ > 1 file changed, 129 insertions(+), 41 deletions(-) > Thanks, Dumitru _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
