Re: [ovs-dev] [PATCH ovn] northd: Use proper field for lookup_nd

2024-01-19 Thread Dumitru Ceara
On 1/19/24 10:28, Xavier Simonart wrote: > Hi Ales > > The patch looks good to me, Thanks! > Acked-by: Xavier Simonart > > Thanks > Xavier > > On Thu, Jan 18, 2024 at 3:54 PM Ales Musil wrote: > >> We are intentionally skipping ND NA with LLA as source. >> However, this doesn't work when the

Re: [ovs-dev] [PATCH ovn] northd: Use proper field for lookup_nd

2024-01-19 Thread Xavier Simonart
Hi Ales The patch looks good to me, Thanks! Acked-by: Xavier Simonart Thanks Xavier On Thu, Jan 18, 2024 at 3:54 PM Ales Musil wrote: > We are intentionally skipping ND NA with LLA as source. > However, this doesn't work when the ND NA has LLA source, > but the target address is global one. I

Re: [ovs-dev] [PATCH ovn] northd: Use proper field for lookup_nd

2024-01-18 Thread 0-day Robot
Bleep bloop. Greetings Ales Musil, 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: The subject summary should end with a dot. Subject: northd: Use proper field for lookup_

[ovs-dev] [PATCH ovn] northd: Use proper field for lookup_nd

2024-01-18 Thread Ales Musil
We are intentionally skipping ND NA with LLA as source. However, this doesn't work when the ND NA has LLA source, but the target address is global one. In that case we would skip update of already existing entry when always_learn_from_arp_request is set to false. Use ND target address in the check