On 2/24/21 10:15 AM, Numan Siddique wrote:
On Tue, Feb 23, 2021 at 6:50 PM Dumitru Ceara <[email protected]> wrote:

Matching on ct_nw_dst()/ct_ipv6_dst()/ct_tp_dst() creates openflows
that often are not offloadable to hardware.  This was used for
detecting load balancer hairpin sessions.

However, it can be avoided if ovn-northd stores the original
destination tuple in OVS registers.  For backwards compatibility,
during upgrade, fall back to matching on the conntrack original tuple.

Signed-off-by: Dumitru Ceara <[email protected]>

Hi Numan,


Acked-by: Numan Siddique <[email protected]>

Thanks for the review!


It will be great if you can enhance the test to pause northd, clear
the lb option
and make sure ovn-controller programs the flow in the old way.

The test I added in patch 1/3 does that already. There's no change in this patch because in backwards compatibility mode flows in table=70 look exactly the same as they did with patch 1/3.


Thanks
Numan


Thanks,
Dumitru

_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to