[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