Re: [ovs-dev] [PATCH v3] ofproto-dpif: APIs and CLI option to add/delete static fdb entry

Hi Eelco, I addressed your comments and also there was a bug uncovered with unit tests. Here is the PATCH v4 Thanks *Vasu Dasari* On Fri, May 21, 2021 at 4:41 AM Eelco Chaudron wrote: > > > On

Re: [ovs-dev] [PATCH v3] ofproto-dpif: APIs and CLI option to add/delete static fdb entry

On 20 May 2021, at 19:09, Vasu Dasari wrote: > Thank you Eelco for testing the patch. > > My responses are inline: Thanks, looking forward for your next revision. Would be good if you can add a test case for the mac move and flush change. //Eelco > *Vasu Dasari* > > > On Thu, May 20, 2021

Re: [ovs-dev] [PATCH v3] ofproto-dpif: APIs and CLI option to add/delete static fdb entry

Thank you Eelco for testing the patch. My responses are inline: *Vasu Dasari* On Thu, May 20, 2021 at 5:20 AM Eelco Chaudron wrote: > > > On 14 May 2021, at 21:33, Vasu Dasari wrote: > > > Currently there is an option to add/flush/show ARP/ND neighbor. This > covers L3 > > side. For L2 side,

Re: [ovs-dev] [PATCH v3] ofproto-dpif: APIs and CLI option to add/delete static fdb entry

On 14 May 2021, at 21:33, Vasu Dasari wrote: > Currently there is an option to add/flush/show ARP/ND neighbor. This covers L3 > side. For L2 side, there is only fdb show command. This patch gives an option > to add/del an fdb entry via ovs-appctl. ovs-appctl command looks like this: > > To add:

[ovs-dev] [PATCH v3] ofproto-dpif: APIs and CLI option to add/delete static fdb entry

Currently there is an option to add/flush/show ARP/ND neighbor. This covers L3 side. For L2 side, there is only fdb show command. This patch gives an option to add/del an fdb entry via ovs-appctl. ovs-appctl command looks like this: To add: ovs-appctl fdb/add ovs-appctl fdb/add br0 p1