Re: [ovs-dev] [PATCH v2 2/2 ovn] OVN: Use ipv4.src and ipv4.dst actions for NAT rules

2019-10-31 Thread Ankur Sharma
Ankur > > -Original Message- > From: Numan Siddique > Sent: Monday, October 28, 2019 6:40 AM > To: Ankur Sharma > Cc: ovs-dev@openvswitch.org > Subject: Re: [ovs-dev] [PATCH v2 2/2 ovn] OVN: Use ipv4.src and > ipv4.dst actions for NAT rules > &

Re: [ovs-dev] [PATCH v2 2/2 ovn] OVN: Use ipv4.src and ipv4.dst actions for NAT rules

2019-10-31 Thread Numan Siddique
Ankur > > -Original Message- > From: Numan Siddique > Sent: Monday, October 28, 2019 6:40 AM > To: Ankur Sharma > Cc: ovs-dev@openvswitch.org > Subject: Re: [ovs-dev] [PATCH v2 2/2 ovn] OVN: Use ipv4.src and ipv4.dst > actions for NAT rules > &

Re: [ovs-dev] [PATCH v2 2/2 ovn] OVN: Use ipv4.src and ipv4.dst actions for NAT rules

2019-10-30 Thread Ankur Sharma
Hi Numan, Submitted a v3 yesterday, addressing all the review comments. Please take a look. Regards, Ankur -Original Message- From: Numan Siddique Sent: Monday, October 28, 2019 6:40 AM To: Ankur Sharma Cc: ovs-dev@openvswitch.org Subject: Re: [ovs-dev] [PATCH v2 2/2 ovn] OVN: Use

Re: [ovs-dev] [PATCH v2 2/2 ovn] OVN: Use ipv4.src and ipv4.dst actions for NAT rules

2019-10-28 Thread Numan Siddique
On Sat, Oct 5, 2019 at 1:45 AM Ankur Sharma wrote: > > For dnat_and_snat rules which are meant to be stateless > instead of using ct_snat/dnat OVN actions, we will use > ipv4.src/ipv4.dst. > > This actions will do 1:1 mapping to inner ip to external ip, > while recalculating the checksums. > >

Re: [ovs-dev] [PATCH v2 2/2 ovn] OVN: Use ipv4.src and ipv4.dst actions for NAT rules

2019-10-04 Thread 0-day Robot
Bleep bloop. Greetings Ankur Sharma, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: Line is 80 characters long (recommended limit is 79) #30 FILE:

[ovs-dev] [PATCH v2 2/2 ovn] OVN: Use ipv4.src and ipv4.dst actions for NAT rules

2019-10-04 Thread Ankur Sharma
For dnat_and_snat rules which are meant to be stateless instead of using ct_snat/dnat OVN actions, we will use ipv4.src/ipv4.dst. This actions will do 1:1 mapping to inner ip to external ip, while recalculating the checksums. Signed-off-by: Ankur Sharma --- northd/ovn-northd.8.xml | 34