Re: [ovs-dev] [PATCH ovn v2 2/2] northd: Use ct_(snat/dnat)_in_czone action for distributed routers.

2021-11-19 Thread Numan Siddique
On Fri, Nov 19, 2021 at 11:16 AM Mark Michelson wrote: > > On 11/18/21 18:42, Numan Siddique wrote: > > On Thu, Nov 18, 2021 at 4:49 PM Mark Michelson wrote: > >> > >> Hi Numan, > >> > >> I had one question in the code and then one small finding in the > >> documentation. See below: > >> > >> On

Re: [ovs-dev] [PATCH ovn v2 2/2] northd: Use ct_(snat/dnat)_in_czone action for distributed routers.

2021-11-19 Thread Mark Michelson
On 11/18/21 18:42, Numan Siddique wrote: On Thu, Nov 18, 2021 at 4:49 PM Mark Michelson wrote: Hi Numan, I had one question in the code and then one small finding in the documentation. See below: On 11/18/21 13:13, num...@ovn.org wrote: From: Numan Siddique Make of use of these new action

Re: [ovs-dev] [PATCH ovn v2 2/2] northd: Use ct_(snat/dnat)_in_czone action for distributed routers.

2021-11-18 Thread Numan Siddique
On Thu, Nov 18, 2021 at 4:49 PM Mark Michelson wrote: > > Hi Numan, > > I had one question in the code and then one small finding in the > documentation. See below: > > On 11/18/21 13:13, num...@ovn.org wrote: > > From: Numan Siddique > > > > Make of use of these new actions for the distributed r

Re: [ovs-dev] [PATCH ovn v2 2/2] northd: Use ct_(snat/dnat)_in_czone action for distributed routers.

2021-11-18 Thread Mark Michelson
Hi Numan, I had one question in the code and then one small finding in the documentation. See below: On 11/18/21 13:13, num...@ovn.org wrote: From: Numan Siddique Make of use of these new actions for the distributed routers for NAT. These new actions ensure that both sNAT and dNAT happens

[ovs-dev] [PATCH ovn v2 2/2] northd: Use ct_(snat/dnat)_in_czone action for distributed routers.

2021-11-18 Thread numans
From: Numan Siddique Make of use of these new actions for the distributed routers for NAT. These new actions ensure that both sNAT and dNAT happens in the same zone. This approach solves a couple of problems: - The datapath flows generated for external traffic which requires dNAT (N -> S)