On Mon, Feb 29, 2016 at 12:53:42PM -0800, Jarno Rajahalme wrote:
> 
> > On Feb 26, 2016, at 4:48 PM, Ben Pfaff <b...@ovn.org> wrote:
> > 
> > On Thu, Feb 25, 2016 at 01:20:17PM -0800, Jarno Rajahalme wrote:
> >> ONF Extension 230 adds support for OpenFlow 1.4 bundles to OpenFlow
> >> 1.3.  Supporting this allows OpenFlow 1.3 controllers to start using
> >> bundles.  Also the ovs-ofctl '--bundle' option can now be used with
> >> OpenFlow 1.3.
> >> 
> >> Signed-off-by: Jarno Rajahalme <ja...@ovn.org>
> > 
> > Thanks for doing this.
> > 
> > This is simpler than I expected, because I thought that there would be
> > at least some messages that differed in some way from OF1.3 to OF1.4.
> > Did you look over the protocol definitions and make sure that they were
> > the same?
> 
> I think it is all the infra you have built to OVS that made this so simple.
> 
> There really are only two message types (control and add message), and
> apart from the extension header they are the same. The message and
> error type enums were the only real difference.

I'm glad to hear that.

> > Thank you for adding all of the tests.
> > 
> > Acked-by: Ben Pfaff <b...@ovn.org>
> 
> Pushed to master,

Great!
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to