On Sat, Feb 27, 2016 at 11:15:58PM +0000, André Mantas wrote: > Ben Pfaff <b...@ovn.org> escreveu no dia sábado, 27/02/2016 às 22:30: > > > On Sat, Feb 27, 2016 at 06:24:23PM +0000, André Mantas wrote: > > > Ok, I'll try to do that. > > > > > > Now related with tests for ofproto.at, if I'm not mistaken, in version > > 2.3 > > > we had the "bundle" command in ovs-ofctl and do things like: > > > > > > ovs-ofctl bundle <bridge> "open_bundle_id=<id>,add_bundle_id=<id> > > > message=<message>[,add_bundle_id=<id> > > > message=<message>...],close_bundle_id=<id>,commit_bundle_id=<id>" > > > > > > Is this not supported any more? The only way to send "bundle-add" > > messages > > > is with with the option --bundle for add-flow, mod-flows, etc? > > > > OVS 2.3 didn't support bundles. ovs-ofctl never supported a "bundle" > > command. > > Oh, ok. I thought it was supported from here: > http://www.pica8.com/document/v2.3/html/ovs-commands-reference/#1081528
Perhaps Pica8 forked OVS for use on their switches, as is their right. However, they have never contributed even a single patch, so I guess they have no desire to be part of the Open vSwitch development community. > > > If so, is the right approach to add a "--bundle" option for the > > packet-out > > > command? > > > > Well, I suppose you could, and perhaps we should to be thorough, but > > packet-out only supports sending packet-outs, so it's going to have > > limited value in practice. > > > > For more usefulness in practice, I guess we could add a more general > > command that accepts a variety of commands. One could think of this as > > an extension of "ovs-ofctl add-flows", which already allows a variety of > > OFPT_FLOW_MODs in its input, although it might make more sense to give > > it a new name. > > So, in which one do you think I should start working right now? I'd be happy with either one to start, but the more general command would be more useful. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
