[ovs-dev] [PATCH monitor_cond V6 11/11] RFC OVN: Quick implementation of conditional monitoring

[Liran Schour]

Darrell Ball <dlu998 at gmail.com> wrote on 18/05/2016 03:56:30 AM:

> On Tue, May 17, 2016 at 7:27 AM, Liran Schour <lirans at il.ibm.com> wrote:
> Conditional monitor of: Port_Binding, Logical_Flow, Multicast_Group
> MAC_Binding tables. As a result ovn-controller will be notified only 
about
> records belongs to a datapath that is being served by this hypervisor.
> 
> Hack: Ideally, logical datapath ID should be retrieved from Port_Binding 
table
> and by that conditions should be composed only from logical datapath 
IDs.
> In the meantime we first add the logical port to the conditions and 
> on retrieval
> of port binding record, we add the logical datapath to the conditions.
> 
> ovn-bridge-mappings unit test fails due to the fact that patch ports 
will not
> be created on local bridge if there is no port binded to the logical
> datapath on
> the local host.
> 
> Is this patch intended to support recursive datapath monitoring ?
> 
> What I mean by this, is if a distributed router (DR) is associated 
> with a set of logical switches (LSs), then
> a HV would filter processing the DR datapath based on whether it has
> at least one LS,
> with local membership, associated with the same DR datapath; i.e. no
> overall membership -> do not process
> the DR datapath on this HV.
> 
> I think the association between DR and associated LSs is already 
> known at the CMS layer.
> 
> Hence,  I am wondering if it is optimal for a CMS to pass down this 
> association directly and 
> then a check done when processing each DR.
> 
> This can also be done for other nested datapath associations, 
> although there seems to be
> an ordering requirement.
> 
> Is this patch series intended to support this goal ?

As I see it a DR is a router and this patch allows each HV 
(ovn-controller) to be updated on each DP (logical switch) it has a local 
vif connected to and all the routed DP's, means the connected router and 
all the connected logical switches.