Thank you Dumitru and Lorenzo. I added the "Reported-at" note to the
final patch of the series.
I pushed the series to main.
On 5/17/22 07:14, Dumitru Ceara wrote:
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
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
