Re: [ovs-dev] [PATCH ovn] Do not reply on unicast arps for targets.

2024-06-03 Thread Vasyl Saienko
I've pushed a new revision, please review. The thread is `[PATCH v2 ovn] Do not reply on unicast arps for IPv4 targets` On Thu, May 23, 2024 at 1:07 PM Han Zhou wrote: > > > On Thu, May 23, 2024 at 1:51 PM Vasyl Saienko > wrote: > > > > Thanks for feedback, > > > > I'm not an ipv6 expert, I

Re: [ovs-dev] [PATCH ovn] Do not reply on unicast arps for targets.

2024-05-23 Thread Han Zhou
On Thu, May 23, 2024 at 1:51 PM Vasyl Saienko wrote: > > Thanks for feedback, > > I'm not an ipv6 expert, I did some research. There is no ARP in ipv6, it's replaced with Neighbor Discovery. The ARP resolution is done via Neighbor Solicitation and Neighbour Advertisement messages. > Neighbor

Re: [ovs-dev] [PATCH ovn] Do not reply on unicast arps for targets.

2024-05-22 Thread Vasyl Saienko
Thanks for feedback, I'm not an ipv6 expert, I did some research. There is no ARP in ipv6, it's replaced with Neighbor Discovery . The ARP resolution is done via Neighbor Solicitation

Re: [ovs-dev] [PATCH ovn] Do not reply on unicast arps for targets.

2024-05-21 Thread Han Zhou
On Tue, May 21, 2024 at 9:56 PM Numan Siddique wrote: > > On Mon, May 20, 2024 at 1:47 AM Vasyl Saienko wrote: > > > > Thanks for feedback Numan. > > > > On Wed, May 15, 2024 at 12:04 AM Numan Siddique wrote: > > > > > On Mon, May 13, 2024 at 9:00 AM Dumitru Ceara wrote: > > > > > > > > On

Re: [ovs-dev] [PATCH ovn] Do not reply on unicast arps for targets.

2024-05-21 Thread Numan Siddique
On Mon, May 20, 2024 at 1:47 AM Vasyl Saienko wrote: > > Thanks for feedback Numan. > > On Wed, May 15, 2024 at 12:04 AM Numan Siddique wrote: > > > On Mon, May 13, 2024 at 9:00 AM Dumitru Ceara wrote: > > > > > > On 5/4/24 11:38, Vasyl Saienko wrote: > > > > Hello Numan > > > > > > > > Thanks

Re: [ovs-dev] [PATCH ovn] Do not reply on unicast arps for targets.

2024-05-19 Thread Vasyl Saienko
Thanks for feedback Numan. On Wed, May 15, 2024 at 12:04 AM Numan Siddique wrote: > On Mon, May 13, 2024 at 9:00 AM Dumitru Ceara wrote: > > > > On 5/4/24 11:38, Vasyl Saienko wrote: > > > Hello Numan > > > > > > Thanks for review and feedback. > > > > > > On Fri, May 3, 2024 at 7:14 PM Numan

Re: [ovs-dev] [PATCH ovn] Do not reply on unicast arps for targets.

2024-05-14 Thread Numan Siddique
On Mon, May 13, 2024 at 9:00 AM Dumitru Ceara wrote: > > On 5/4/24 11:38, Vasyl Saienko wrote: > > Hello Numan > > > > Thanks for review and feedback. > > > > On Fri, May 3, 2024 at 7:14 PM Numan Siddique wrote: > > > >> On Mon, Apr 22, 2024 at 2:54 AM Vasyl Saienko > >> wrote: > >>> > >>>

Re: [ovs-dev] [PATCH ovn] Do not reply on unicast arps for targets.

2024-05-13 Thread Dumitru Ceara
On 5/4/24 11:38, Vasyl Saienko wrote: > Hello Numan > > Thanks for review and feedback. > > On Fri, May 3, 2024 at 7:14 PM Numan Siddique wrote: > >> On Mon, Apr 22, 2024 at 2:54 AM Vasyl Saienko >> wrote: >>> >>> Reply only if target ethernet address is broadcast, if >>> address is specified

Re: [ovs-dev] [PATCH ovn] Do not reply on unicast arps for targets.

2024-05-04 Thread Vasyl Saienko
Hello Numan Thanks for review and feedback. On Fri, May 3, 2024 at 7:14 PM Numan Siddique wrote: > On Mon, Apr 22, 2024 at 2:54 AM Vasyl Saienko > wrote: > > > > Reply only if target ethernet address is broadcast, if > > address is specified explicitly do noting to let target > > reply by

Re: [ovs-dev] [PATCH ovn] Do not reply on unicast arps for targets.

2024-05-03 Thread Numan Siddique
On Mon, Apr 22, 2024 at 2:54 AM Vasyl Saienko wrote: > > Reply only if target ethernet address is broadcast, if > address is specified explicitly do noting to let target > reply by itself. This technique allows to monitor target > aliveness with arping. Thanks for the patch. This patch does

[ovs-dev] [PATCH ovn] Do not reply on unicast arps for targets.

2024-04-22 Thread Vasyl Saienko
Reply only if target ethernet address is broadcast, if address is specified explicitly do noting to let target reply by itself. This technique allows to monitor target aliveness with arping. Closes #239 Signed-off-by: Vasyl Saienko --- northd/northd.c | 11 +-- tests/ovn-northd.at