Re: [ovs-dev] [PATCH ovn 2/2] ovn-controller: Assume well-known tables are in the SB schema.

2023-08-17 Thread Vladislav Odintsov
Thanks Dumitru for the efforts! > On 15 Aug 2023, at 13:39, Dumitru Ceara wrote: > > On 8/14/23 00:24, Han Zhou wrote: >> On Fri, Aug 11, 2023 at 5:25 AM Dumitru Ceara wrote: >>> >>> It's safe to assume that tables that existed in the previous LTS branch >>> first release (currently 22.03.0) c

Re: [ovs-dev] [PATCH ovn 2/2] ovn-controller: Assume well-known tables are in the SB schema.

2023-08-15 Thread Dumitru Ceara
On 8/14/23 00:24, Han Zhou wrote: > On Fri, Aug 11, 2023 at 5:25 AM Dumitru Ceara wrote: >> >> It's safe to assume that tables that existed in the previous LTS branch >> first release (currently 22.03.0) can be monitored directly. Do so and >> only "optionally" monitor the ones that have been add

Re: [ovs-dev] [PATCH ovn 2/2] ovn-controller: Assume well-known tables are in the SB schema.

2023-08-13 Thread Ales Musil
On Fri, Aug 11, 2023 at 2:25 PM Dumitru Ceara wrote: > It's safe to assume that tables that existed in the previous LTS branch > first release (currently 22.03.0) can be monitored directly. Do so and > only "optionally" monitor the ones that have been added since. > > This way we avoid the need

Re: [ovs-dev] [PATCH ovn 2/2] ovn-controller: Assume well-known tables are in the SB schema.

2023-08-13 Thread Han Zhou
On Fri, Aug 11, 2023 at 5:25 AM Dumitru Ceara wrote: > > It's safe to assume that tables that existed in the previous LTS branch > first release (currently 22.03.0) can be monitored directly. Do so and > only "optionally" monitor the ones that have been added since. > > This way we avoid the need

[ovs-dev] [PATCH ovn 2/2] ovn-controller: Assume well-known tables are in the SB schema.

2023-08-11 Thread Dumitru Ceara
It's safe to assume that tables that existed in the previous LTS branch first release (currently 22.03.0) can be monitored directly. Do so and only "optionally" monitor the ones that have been added since. This way we avoid the need for the IDL to expose an API to change the default condition for