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
>
&
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
>
&
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
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.
>
>
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:
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