Re: [ovs-dev] [Patch ovn v2 2/2] northd: Fix direct access to SNAT network on DR.

2024-04-09 Thread Martin Kalcok
Hi Ales, thanks for another round of review. On Wed, Mar 20, 2024 at 12:34 PM Ales Musil wrote: > > > On Thu, Mar 14, 2024 at 2:13 PM Martin Kalcok > wrote: > >> Hello all, >> I have one more follow-up regarding the comments in v1. @amusil, you were >> concerned about the impact this change

Re: [ovs-dev] [Patch ovn v2 2/2] northd: Fix direct access to SNAT network on DR.

2024-03-20 Thread Ales Musil
On Thu, Mar 14, 2024 at 2:13 PM Martin Kalcok wrote: > Hello all, > I have one more follow-up regarding the comments in v1. @amusil, you were > concerned about the impact this change would have on the performance so I > ran some tests to try to gauge it. I used following setup with two physical

Re: [ovs-dev] [Patch ovn v2 2/2] northd: Fix direct access to SNAT network on DR.

2024-03-14 Thread Martin Kalcok
Hello all, I have one more follow-up regarding the comments in v1. @amusil, you were concerned about the impact this change would have on the performance so I ran some tests to try to gauge it. I used following setup with two physical hosts connected over switch: |Physical ext. |

Re: [ovs-dev] [Patch ovn v2 2/2] northd: Fix direct access to SNAT network on DR.

2024-03-13 Thread Martin Kalcok
Regarding the failed unstable test in the CI, I suspect that this is not related to the code change, I've had couple successful CI runs in my branch [0]. Martin. [0] https://github.com/mkalcok/ovn/actions/runs/8256539983 On Tue, Mar 12, 2024 at 8:45 PM Martin Kalcok wrote: > This change fixes

[ovs-dev] [Patch ovn v2 2/2] northd: Fix direct access to SNAT network on DR.

2024-03-12 Thread Martin Kalcok
This change fixes bug that breaks ability of machines from external networks to communicate with machines in SNATed networks (specifically when using a Distributed router). Currently when a machine (S1) on an external network tries to talk over TCP with a machine (A1) in a network that has