Re: [ovs-dev] [PATCH ovn v3] northd: forward arp request to lrp snat on.

2023-12-08 Thread Dumitru Ceara
On 12/8/23 14:51, Daniel Ding wrote: > On Thu, 7 Dec 2023 at 21:26, Dumitru Ceara wrote: > >> On 12/6/23 02:56, Daniel Ding wrote: >>> Hi Dumitru! >>> >>> On Tue, 5 Dec 2023 at 23:59, Dumitru Ceara wrote: >>> On 12/5/23 13:58, Daniel Ding wrote: > > > On Tue, 5 Dec 2023 at

Re: [ovs-dev] [PATCH ovn v3] northd: forward arp request to lrp snat on.

2023-12-08 Thread Daniel Ding
On Thu, 7 Dec 2023 at 21:26, Dumitru Ceara wrote: > On 12/6/23 02:56, Daniel Ding wrote: > > Hi Dumitru! > > > > On Tue, 5 Dec 2023 at 23:59, Dumitru Ceara wrote: > > > >> On 12/5/23 13:58, Daniel Ding wrote: > >>> > >>> > >>> On Tue, 5 Dec 2023 at 18:58, Dumitru Ceara >>>

Re: [ovs-dev] [PATCH ovn v3] northd: forward arp request to lrp snat on.

2023-12-07 Thread Dumitru Ceara
On 12/6/23 02:56, Daniel Ding wrote: > Hi Dumitru! > > On Tue, 5 Dec 2023 at 23:59, Dumitru Ceara wrote: > >> On 12/5/23 13:58, Daniel Ding wrote: >>> >>> >>> On Tue, 5 Dec 2023 at 18:58, Dumitru Ceara >> > wrote: >>> >>> Hi Daniel, >>> >>> Thanks for this new

Re: [ovs-dev] [PATCH ovn v3] northd: forward arp request to lrp snat on.

2023-12-05 Thread Daniel Ding
Hi Dumitru! On Tue, 5 Dec 2023 at 23:59, Dumitru Ceara wrote: > On 12/5/23 13:58, Daniel Ding wrote: > > > > > > On Tue, 5 Dec 2023 at 18:58, Dumitru Ceara > > wrote: > > > > Hi Daniel, > > > > Thanks for this new revision but why is it v3? I don't think I

Re: [ovs-dev] [PATCH ovn v3] northd: forward arp request to lrp snat on.

2023-12-05 Thread Dumitru Ceara
On 12/5/23 13:58, Daniel Ding wrote: > > > On Tue, 5 Dec 2023 at 18:58, Dumitru Ceara > wrote: > > Hi Daniel, > > Thanks for this new revision but why is it v3?  I don't think I saw v2 > posted anywhere but maybe I missed it. > >   > Sorry, that is my

Re: [ovs-dev] [PATCH ovn v3] northd: forward arp request to lrp snat on.

2023-12-05 Thread Daniel Ding
On Tue, 5 Dec 2023 at 18:58, Dumitru Ceara wrote: > Hi Daniel, > > Thanks for this new revision but why is it v3? I don't think I saw v2 > posted anywhere but maybe I missed it. > > Sorry, that is my mistake. > On 12/5/23 08:33, Daniel Ding wrote: > > If the router has a snat rule and it's

Re: [ovs-dev] [PATCH ovn v3] northd: forward arp request to lrp snat on.

2023-12-05 Thread Dumitru Ceara
Hi Daniel, Thanks for this new revision but why is it v3? I don't think I saw v2 posted anywhere but maybe I missed it. On 12/5/23 08:33, Daniel Ding wrote: > If the router has a snat rule and it's external ip isn't lrp address, > when the arp request from other router for this external ip,

[ovs-dev] [PATCH ovn v3] northd: forward arp request to lrp snat on.

2023-12-04 Thread Daniel Ding
If the router has a snat rule and it's external ip isn't lrp address, when the arp request from other router for this external ip, will be drop, because of this external ip use same mac address as lrp, so can not forward to MC_FLOOD. Fixes: 32f5ebb06226 ("ovn-northd: Limit ARP/ND broadcast domain