[ovs-dev] [PATCH ovn] ovn-northd: Forward ARP requests on localnet ports.

Dumitru Ceara dceara at redhat.com
Mon Mar 23 12:36:40 UTC 2020


On 3/23/20 1:29 PM, Numan Siddique wrote:
> On Mon, Mar 23, 2020 at 5:53 PM Dumitru Ceara <dceara at redhat.com> wrote:
>>
>> Commit 32f5ebb06226 limited the ARP/ND broadcast domain but in scenarios
>> where ARP responder flows are installed only on chassis that own the
>> associated logical ports ARP requests should still be forwarded on
>> localnet ports because the router pipeline should be executed on the
>> chassis that owns the logical port. Only that chassis will reply to the
>> ARP/ND request.
>>
>> Reported-by: Michael Plato <michael.plato at tu-berlin.de>
>> Reported-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2020-March/049856.html
>> Fixes: 32f5ebb06226 ("ovn-northd: Limit ARP/ND broadcast domain whenever possible.")
>> Signed-off-by: Dumitru Ceara <dceara at redhat.com>
> 
> Can you please update the ovn-northd.8.xml for the modified logical flow.
> 
> Thanks
> Numan

Hi Numan,

Oops, thanks for pointing it out. I updated the doc in v2:
https://patchwork.ozlabs.org/patch/1259989/

Thanks,
Dumitru

> 
>> ---
>>  northd/ovn-northd.c | 6 +++++-
>>  1 file changed, 5 insertions(+), 1 deletion(-)
>>
>> diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c
>> index f648d2e..b76df05 100644
>> --- a/northd/ovn-northd.c
>> +++ b/northd/ovn-northd.c
>> @@ -5903,8 +5903,12 @@ build_lswitch_rport_arp_req_flow_for_ip(struct sset *ips,
>>      ds_put_cstr(&match, "}");
>>
>>      /* Send a the packet only to the router pipeline and skip flooding it
>> -     * in the broadcast domain.
>> +     * in the broadcast domain (except for the localnet port).
>>       */
>> +    if (od->localnet_port) {
>> +        ds_put_format(&actions, "clone { outport = %s; output; }; ",
>> +                      od->localnet_port->json_key);
>> +    }
>>      ds_put_format(&actions, "outport = %s; output;", patch_op->json_key);
>>      ovn_lflow_add_with_hint(lflows, od, S_SWITCH_IN_L2_LKUP, priority,
>>                              ds_cstr(&match), ds_cstr(&actions), stage_hint);
>> --
>> 1.8.3.1
>>
>> _______________________________________________
>> dev mailing list
>> dev at openvswitch.org
>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>>
> 



More information about the dev mailing list