Re: [ovs-dev] [PATCH ovn 0/5] Add MAC binding aging mechanism

2022-06-15 Thread Dumitru Ceara
Hi, Ales, Daniel, On 6/15/22 14:36, Daniel Alvarez Sanchez wrote: > On Wed, Jun 15, 2022 at 1:54 PM Ales Musil wrote: > >> Hi Daniel, >> >> thank you for the suggestions, please see my replies inline. >> >> On Wed, Jun 15, 2022 at 1:00 PM Daniel Alvarez Sanchez < >> dalva...@redhat.com> wrote:

Re: [ovs-dev] [PATCH ovn 0/5] Add MAC binding aging mechanism

2022-06-15 Thread Daniel Alvarez Sanchez
On Wed, Jun 15, 2022 at 1:54 PM Ales Musil wrote: > Hi Daniel, > > thank you for the suggestions, please see my replies inline. > > On Wed, Jun 15, 2022 at 1:00 PM Daniel Alvarez Sanchez < > dalva...@redhat.com> wrote: > >> Hey Ales, first of all thanks a lot for this series!! Great job and good

Re: [ovs-dev] [PATCH ovn 0/5] Add MAC binding aging mechanism

2022-06-15 Thread Ales Musil
Hi Daniel, thank you for the suggestions, please see my replies inline. On Wed, Jun 15, 2022 at 1:00 PM Daniel Alvarez Sanchez wrote: > Hey Ales, first of all thanks a lot for this series!! Great job and good > approach. > > I however have some concerns but maybe they're not much of a deal,

Re: [ovs-dev] [PATCH ovn 0/5] Add MAC binding aging mechanism

2022-06-15 Thread Daniel Alvarez Sanchez
Hey Ales, first of all thanks a lot for this series!! Great job and good approach. I however have some concerns but maybe they're not much of a deal, please let me know what you think: - Broadcast gARPs will cause all ovn-controller instances in the cluster to attempt to write into the SB

[ovs-dev] [PATCH ovn 0/5] Add MAC binding aging mechanism

2022-06-14 Thread Ales Musil
Add MAC binding aging mechanism, that should take care of stale MAC bindings. The mechanism works on "ownership" of the MAC binding row. The chassis that creates the row is then checking if the "idle_age" of the flow is over the aging threshold. In that case the MAC binding is removed from