Re: [ovs-dev] [PATCH ovn] northd: Use separate SNAT for already-DNATted traffic.

2022-03-24 Thread Numan Siddique
On Thu, Mar 24, 2022 at 4:47 PM Mark Michelson wrote: > > Hi Numan, > > This message got buried in my inbox, so sorry about only replying now. I > have some replies in-line below > > On 3/18/22 16:34, Numan Siddique wrote: > > On Wed, Mar 9, 2022 at 2:38 PM Mark Michelson wrote: > >> > >> Commit

Re: [ovs-dev] [PATCH ovn] northd: Use separate SNAT for already-DNATted traffic.

2022-03-24 Thread Mark Michelson
Hi Numan, This message got buried in my inbox, so sorry about only replying now. I have some replies in-line below On 3/18/22 16:34, Numan Siddique wrote: On Wed, Mar 9, 2022 at 2:38 PM Mark Michelson wrote: Commit 4deac4509abbedd6ffaecf27eed01ddefccea40a introduced functionality in ovn-no

Re: [ovs-dev] [PATCH ovn] northd: Use separate SNAT for already-DNATted traffic.

2022-03-18 Thread Numan Siddique
On Wed, Mar 9, 2022 at 2:38 PM Mark Michelson wrote: > > Commit 4deac4509abbedd6ffaecf27eed01ddefccea40a introduced functionality > in ovn-northd to use a single zone for SNAT and DNAT when possible. It > accounts for certain situations, such as hairpinned traffic, where we > still need a separate

[ovs-dev] [PATCH ovn] northd: Use separate SNAT for already-DNATted traffic.

2022-03-09 Thread Mark Michelson
Commit 4deac4509abbedd6ffaecf27eed01ddefccea40a introduced functionality in ovn-northd to use a single zone for SNAT and DNAT when possible. It accounts for certain situations, such as hairpinned traffic, where we still need a separate SNAT and DNAT zone. However, one situation it did not account