> On Oct 7, 2015, at 1:57 PM, Ben Pfaff <b...@nicira.com> wrote: > > +# Waits until the OVS database contains exactly the specified patch ports. > +# Each argument should be of the form BRIDGE PORT PEER. > +check_patches () { > + # Generate code to check that the set of patch ports is exactly as > + # specified. > + printf > query 'ovs-vsctl -f csv -d bare --no-headings --columns=name > find interface type=patch | sort > +ovs-vsctl init'
I think it would be helpful to explain why "ovs-vsctl init" is being used. I got hung up on "ovs-vsctl init" and not the fact that it was building an ovs-vsctl command with a variable number of arguments, which could be zero, in which case the "init" is a no-op that allows the command to run without generating an error. Acked-by: Justin Pettit <jpet...@nicira.com> --Justin _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev