Thanks for the review I'll merge this shortly
On Thu, Jun 7, 2012 at 5:28 PM, Ben Pfaff <b...@nicira.com> wrote: > On Thu, Jun 07, 2012 at 05:27:10PM -0700, Ethan Jackson wrote: >> > Possibly another phrasing would be more user-friendly in lacp/show >> > output, such as "may be enabled: true/false" or "may [not] be enabled". >> >> I'd prefer to keep the current phrasing because it's exposed with the >> same formatting in the ovs-appctl bond/show output as well to mean the >> same variable (pulled from the lacp module). We could, of course, >> change it in both places, but I'm not sure if it's worth it. If you >> think it is I can write up a patch. > > I didn't remember there was precedent. It's fine, then. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
