Re: [ovs-dev] [PATCH ovn] ovn-controller to no longer monitor Chassis' external_ids

2020-05-12 Thread Lucas Alvares Gomes
Thank you all for looking into this. On Tue, May 12, 2020 at 1:37 PM Dumitru Ceara wrote: > > On 5/7/20 12:12 PM, lmart...@redhat.com wrote: > > From: Lucas Alvares Gomes > > > > Prior to this patch, the external_ids column from the Chassis table were > > being used for two purposes: > > > > 1.

Re: [ovs-dev] [PATCH ovn] ovn-controller to no longer monitor Chassis' external_ids

2020-05-12 Thread Dumitru Ceara
On 5/7/20 12:12 PM, lmart...@redhat.com wrote: > From: Lucas Alvares Gomes > > Prior to this patch, the external_ids column from the Chassis table were > being used for two purposes: > > 1. Holding configuration for the OVN (copied by ovn-controller from the > local OVS database) > > 2.

Re: [ovs-dev] [PATCH ovn] ovn-controller to no longer monitor Chassis' external_ids

2020-05-07 Thread Lucas Alvares Gomes
Hi, On Thu, May 7, 2020 at 12:09 PM Daniel Alvarez Sanchez wrote: > > Thanks Lucas! > > The only problem I see is that we still need something like the > Private_Chassis approach that we discussed or otherwise bumping nb_cfg will > still be generating N notifications (N == num_chassis) on every

Re: [ovs-dev] [PATCH ovn] ovn-controller to no longer monitor Chassis' external_ids

2020-05-07 Thread Daniel Alvarez Sanchez
Thanks Lucas! The only problem I see is that we still need something like the Private_Chassis approach that we discussed or otherwise bumping nb_cfg will still be generating N notifications (N == num_chassis) on every write. Still, this solves part of the problem. I was wondering if we could

[ovs-dev] [PATCH ovn] ovn-controller to no longer monitor Chassis' external_ids

2020-05-07 Thread lmartins
From: Lucas Alvares Gomes Prior to this patch, the external_ids column from the Chassis table were being used for two purposes: 1. Holding configuration for the OVN (copied by ovn-controller from the local OVS database) 2. Holding information from external systems (the main purpose of the