[ovs-dev] [cfm 3/7] cfm: Remove Maintenance_Point and Monitor tables.

Ben Pfaff blp at nicira.com
Fri May 20 21:37:21 UTC 2011


On Fri, May 20, 2011 at 02:32:41PM -0700, Ethan Jackson wrote:
> > I don't understand the last point there. ?The fault status of the
> > maintenance point is still available from the Interface column
> > cfm_fault, isn't it? ?What was lost?
> 
> This could be worded better in the commit message.  Previously, you
> were able to configure multiple remote maintenance points, and check
> the fault status of each of those maintenance points individually in
> addition to the CFM object's fault status.  Now you can only check the
> CFM object's fault status.  Since we only allow one remote maintenance
> point, you can infer the fault status of the remote maintenance point
> from that, but you can't read it directly.

OK.  Thanks for clarifying.

> > I see that the internal CFM code still supports multiple remote MPs,
> > even though the database and ofproto interfaces can only configure a
> > single one. ?Any particular reason?
> 
> I'd kinda wanted to delete the support.  Justin thought we should keep
> it around in case we ever need support for multiple remote maintenance
> points again in the future.  I'm more or less neutral on the subject I
> suppose.

Not a big deal.  This way is fine.



More information about the dev mailing list