Re: [ovs-dev] [patch v2] dpctl: Simplify dpctl_flush_conntrack.

2018-08-06 Thread Darrell Ball
thanks for checking that; I thought the system tests has full coverage for these aspects. The proposed change would cause inappropriate log entries. Anyways, I rethought the approach to make it more general and sent a V3. On 8/3/18, 5:36 PM, "ovs-dev-boun...@openvswitch.org on behalf of Ben

Re: [ovs-dev] [patch v2] dpctl: Simplify dpctl_flush_conntrack.

2018-08-03 Thread Ben Pfaff
On Thu, Aug 02, 2018 at 08:20:58PM -0700, Darrell Ball wrote: > The function dpctl_flush_conntrack() and other such functions with > multiple optional arguments can be simplified by introducing a new > function to check whether a valid datapath name is supplied as an > argument to the functions. >

[ovs-dev] [patch v2] dpctl: Simplify dpctl_flush_conntrack.

2018-08-02 Thread Darrell Ball
The function dpctl_flush_conntrack() and other such functions with multiple optional arguments can be simplified by introducing a new function to check whether a valid datapath name is supplied as an argument to the functions. opt_dpif_open() can also make use of this new function to allow it to