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

[Kyle Mestery]

On Mon, Nov 23, 2015 at 12:52 PM, Russell Bryant <russell at ovn.org> wrote:

>
>
> On Mon, Nov 23, 2015 at 1:46 PM, Justin Pettit <jpettit at ovn.org> wrote:
>
>>
>> > On Nov 23, 2015, at 10:27 AM, Russell Bryant <russell at ovn.org> wrote:
>> >
>> > 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.  :-)
>>
>> I suspect it was the "of", not the "BDF".  :-)
>>
>
> Ah ha!  That's the change I was looking for but couldn't spot.  :-)
>
> Sorry for being pedantic, but Justin called it, it was the "of." :)


> --
> Russell Bryant
>