OK, I'll apply it in a minute. Also, I'm taking the liberty of interpreting your reply as an Acked-by.
Thanks, Ben. On Wed, May 07, 2014 at 11:24:43AM -0700, Bruce Davie wrote: > Ben, > Thanks for the review. We have not yet tested any implementations of BFD on > VTEPs, so I?m not too worried about backward compatibility. We discovered the > need for this change to BFD in the schema when looking at how we would > configure BFD tunnels terminating on several VTEPs managed by a single HSC. > I?ve reviewed Ashwin?s patch and believe it is a good design to address that > issue. > > Summary - I?m fine with this patch being applied. > > Bruce > > On May 7, 2014, at 11:17 AM, Ben Pfaff <b...@nicira.com> wrote: > > > On Tue, May 06, 2014 at 02:54:52PM -0700, Ashwin Swaminathan wrote: > >> Added a Tunnel table to the VTEP schema that allows > >> per-tunnel BFD configuration and status to be specified. > >> Removed the BFD configuration/status from the > >> Physical_Locator table. > >> > >> Signed-off-by: Ashwin Swaminathan <ashwi...@arista.com> > > > > This is a non backward compatible change, so it could cause problems > > for vendors who implemented the previous schema. I don't have > > information on how many vendors that affects, in practice (none, I > > hope). > > > > Adding Bruce, who might better know the implications. Bruce, can you > > tell us anything? > > > > Thanks, > > > > Ben. > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
