Re: [ovs-dev] [PATCH, v2] ovn: Allow SNAT traffic destined to router ip

2016-07-18 Thread Chandra Sekhar Vejendla
Ryan Moats/Omaha/IBM wrote on 07/18/2016 10:03:35 AM: > From: Ryan Moats/Omaha/IBM > To: Chandra Sekhar Vejendla/San Jose/IBM@IBMUS > Cc: Guru Shetty , ovs dev > Date: 07/18/2016 10:03 AM > Subject: Re: [ovs-dev] [PATCH, v2] ovn: Allow SNAT traffic destined > to router ip &

Re: [ovs-dev] [PATCH, v2] ovn: Allow SNAT traffic destined to router ip

2016-07-18 Thread Ryan Moats
"dev" wrote on 07/18/2016 11:20:59 AM: > From: Chandra Sekhar Vejendla/San Jose/IBM@IBMUS > To: Guru Shetty > Cc: ovs dev > Date: 07/18/2016 11:21 AM > Subject: Re: [ovs-dev] [PATCH, v2] ovn: Allow SNAT traffic destined > to router ip > Sent by: "dev"

Re: [ovs-dev] [PATCH, v2] ovn: Allow SNAT traffic destined to router ip

2016-07-18 Thread Chandra Sekhar Vejendla
Guru Shetty wrote on 07/18/2016 08:03:52 AM: > From: Guru Shetty > To: Chandra Sekhar Vejendla/San Jose/IBM@IBMUS > Cc: ovs dev > Date: 07/18/2016 08:04 AM > Subject: Re: [ovs-dev] [PATCH, v2] ovn: Allow SNAT traffic destined > to router ip > > On 18 July 2016 at 0

Re: [ovs-dev] [PATCH, v2] ovn: Allow SNAT traffic destined to router ip

2016-07-18 Thread Guru Shetty
On 18 July 2016 at 05:22, Chandra S Vejendla wrote: > When router ip is used as SNAT IP, traffic destined to router > ip should not be dropped > Thank you for the fix. You will need to add your Signed-off-by. Can you also add a "Fixes:" tag in commit message. Since this is a regression, I wonder

[ovs-dev] [PATCH, v2] ovn: Allow SNAT traffic destined to router ip

2016-07-18 Thread Chandra S Vejendla
When router ip is used as SNAT IP, traffic destined to router ip should not be dropped --- ovn/northd/ovn-northd.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c index 7ce509d..78c3a7d 100644 --- a/ovn/northd/ovn-northd.c