On Tue, May 17, 2016 at 05:27:09PM +0300, Liran Schour 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. > > Signed-off-by: Liran Schour <lir...@il.ibm.com>
Looking at how all this fits together, it seems a little awkward to have condition objects separate from the IDL and to have them require a separate "update" call. It would be easier to use and probably more straightforward if it wasn't necessary to have (and track) these separate objects, and then conditionally update them. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
