> 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

Reply via email to