Re: [ovs-dev] [PATCH ovn v3 2/2] northd, controller: Use paused controller action for packet buffering.

2024-04-16 Thread Ales Musil
On Tue, Apr 16, 2024 at 9:27 PM Mark Michelson wrote: > Hi Ales, I have a couple of notes below. > Hi Mark, thank you for the review. > > On 4/10/24 09:29, Ales Musil wrote: > > The current packet injection loses ct_state in the process. When > > the ct_state is lost we might commit to DNAT

Re: [ovs-dev] [PATCH ovn v3 2/2] northd, controller: Use paused controller action for packet buffering.

2024-04-16 Thread Mark Michelson
Hi Ales, I have a couple of notes below. On 4/10/24 09:29, Ales Musil wrote: The current packet injection loses ct_state in the process. When the ct_state is lost we might commit to DNAT zone and perform zero SNAT after the packet injection. This causes the first session to be wrong as the

[ovs-dev] [PATCH ovn v3 2/2] northd, controller: Use paused controller action for packet buffering.

2024-04-10 Thread Ales Musil
The current packet injection loses ct_state in the process. When the ct_state is lost we might commit to DNAT zone and perform zero SNAT after the packet injection. This causes the first session to be wrong as the reply packets are not unDNATted. Instead of re-injecting the packet back into the