On Mon, Nov 23, 2015 at 12:52 PM, Russell Bryant <russ...@ovn.org> wrote:
> > > On Mon, Nov 23, 2015 at 1:46 PM, Justin Pettit <jpet...@ovn.org> wrote: > >> >> > On Nov 23, 2015, at 10:27 AM, Russell Bryant <russ...@ovn.org> wrote: >> > >> > On 11/23/2015 11:36 AM, Kyle Mestery wrote: >> >> On Mon, Nov 23, 2015 at 10:25 AM, Russell Bryant <russ...@ovn.org >> >> <mailto:russ...@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 <russ...@ovn.org <mailto: >> russ...@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 <mest...@mestery.com <mailto: >> mest...@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 > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
