Re: [ovs-dev] [PATCH ovn v2] northd: Provide the Gateway router option 'lb_force_snat_ip' to take router port ips.

2021-02-18 Thread Numan Siddique
On Thu, Feb 18, 2021 at 10:57 PM Mark Gray wrote: > > On 18/02/2021 17:21, Numan Siddique wrote: > > On Thu, Feb 18, 2021 at 9:08 PM Mark Gray wrote: > >> > >> On 18/02/2021 14:30, num...@ovn.org wrote: > >>> From: Numan Siddique > >>> > >>> When a Gateway router is configured with a load

Re: [ovs-dev] [PATCH ovn v2] northd: Provide the Gateway router option 'lb_force_snat_ip' to take router port ips.

2021-02-18 Thread Mark Gray
On 18/02/2021 17:21, Numan Siddique wrote: > On Thu, Feb 18, 2021 at 9:08 PM Mark Gray wrote: >> >> On 18/02/2021 14:30, num...@ovn.org wrote: >>> From: Numan Siddique >>> >>> When a Gateway router is configured with a load balancer >>> and it is also configured with options:lb_force_snat_ip=,

Re: [ovs-dev] [PATCH ovn v2] northd: Provide the Gateway router option 'lb_force_snat_ip' to take router port ips.

2021-02-18 Thread Numan Siddique
On Thu, Feb 18, 2021 at 9:08 PM Mark Gray wrote: > > On 18/02/2021 14:30, num...@ovn.org wrote: > > From: Numan Siddique > > > > When a Gateway router is configured with a load balancer > > and it is also configured with options:lb_force_snat_ip=, > > OVN after load balancing the destination IP

Re: [ovs-dev] [PATCH ovn v2] northd: Provide the Gateway router option 'lb_force_snat_ip' to take router port ips.

2021-02-18 Thread Mark Gray
On 18/02/2021 14:30, num...@ovn.org wrote: > From: Numan Siddique > > When a Gateway router is configured with a load balancer > and it is also configured with options:lb_force_snat_ip=, > OVN after load balancing the destination IP to one of the > backend also does a NAT on the source ip with

Re: [ovs-dev] [PATCH ovn v2] northd: Provide the Gateway router option 'lb_force_snat_ip' to take router port ips.

2021-02-18 Thread Numan Siddique
On Thu, Feb 18, 2021 at 8:31 PM 0-day Robot wrote: > > Bleep bloop. Greetings Numan Siddique, 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 94

Re: [ovs-dev] [PATCH ovn v2] northd: Provide the Gateway router option 'lb_force_snat_ip' to take router port ips.

2021-02-18 Thread 0-day Robot
Bleep bloop. Greetings Numan Siddique, 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 94 characters long (recommended limit is 79) WARNING: Line has trailing

[ovs-dev] [PATCH ovn v2] northd: Provide the Gateway router option 'lb_force_snat_ip' to take router port ips.

2021-02-18 Thread numans
From: Numan Siddique When a Gateway router is configured with a load balancer and it is also configured with options:lb_force_snat_ip=, OVN after load balancing the destination IP to one of the backend also does a NAT on the source ip with the lb_force_snat_ip if the packet is destined to a load