[ovs-dev] [PATCH] ovn: Update BFD item in the ovn TODO list.

Russell Bryant russell at ovn.org
Mon Nov 23 18:27:02 UTC 2015


On 11/23/2015 11:36 AM, Kyle Mestery wrote:
> On Mon, Nov 23, 2015 at 10:25 AM, Russell Bryant <russell at ovn.org
> <mailto:russell at ovn.org>> wrote:
> 
>     Update the BFD todo item to clarify where we might use BFD as it
>     previously seemed to imply we wanted to enable it for all
>     hypervisor-to-hypervisor tunnels.
> 
>     Signed-off-by: Russell Bryant <russell at ovn.org <mailto:russell at ovn.org>>
>     ---
> 
> 
>     This patch attemptes to update the TODO item based on our ML discussion.
>     Alternatively, we could just remove the TODO item since it's more
>     speculative
>     at this point.
> 
>      ovn/TODO | 15 +++++++++++----
>      1 file changed, 11 insertions(+), 4 deletions(-)
> 
> 
>     diff --git a/ovn/TODO b/ovn/TODO
>     index 1f2a73f..22d6bd8 100644
>     --- a/ovn/TODO
>     +++ b/ovn/TODO
>     @@ -306,11 +306,18 @@ think it does everything else.
>         the Multicast_Group table entry in ovn-sb database into
>         Mcast_Macs_Remote table configuration in VTEP database.
> 
>     -* Use BFD as tunnel monitor.
>     +* Consider the use BFD as tunnel monitor.
> 
> This reads odd to me. Maybe something like this:
> 
> "Consider the use of BDF as tunnel monitor."
> 
> With that change:
> 
> Acked-By: Kyle Mestery <mestery at mestery.com <mailto:mestery at mestery.com>>

Did you mean to change it further?  The only change I see is from "BFD"
to "BDF", which I don't think was intentional.  :-)

-- 
Russell Bryant



More information about the dev mailing list