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

Matching on ct.dnat creates openflows that often are not offloadable
to hardware.  ovn-northd uses ct.dnat only for load balancer hairpin
traffic handling and it turns out we don't really need to match on
ct.dnat.

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

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

Can you please add tests in ovn-northd for this ? This would make sure
ovn-northd-ddlog (once we have the ddlog patches)
is not out of sync with northd.

Good idea, thanks for pointing it out!

+Ben

I think that if my patches go in before the ddlog ones it's still ok to accept the ddlog ones in their current state. I can work on a follow up to add ddlog support for the new LB hairpin logical flows soon afterwards.


Thanks
Numan


Thanks,
Dumitru

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

Reply via email to