Re: [ovs-dev] [PATCH 2/4] ovs-vsctl: Use default socket name in tests.

2018-02-06 Thread Ben Pfaff
Thanks! I applied this series to master. On Tue, Feb 06, 2018 at 11:05:16AM -0800, Yifeng Sun wrote: > Thanks. > > Tested-by: Yifeng Sun > > Reviewed-by: Yifeng Sun > > On Tue, Feb 6, 2018 at 10:15 AM, Ben Pfaff wrote: > > >

Re: [ovs-dev] [PATCH 2/4] ovs-vsctl: Use default socket name in tests.

2018-02-06 Thread Yifeng Sun
Thanks. Tested-by: Yifeng Sun Reviewed-by: Yifeng Sun On Tue, Feb 6, 2018 at 10:15 AM, Ben Pfaff wrote: > Thanks for pointing that out. The following incremental patch fixes the > problem. I will fold it in: > > diff --git

Re: [ovs-dev] [PATCH 2/4] ovs-vsctl: Use default socket name in tests.

2018-02-06 Thread Ben Pfaff
Thanks for pointing that out. The following incremental patch fixes the problem. I will fold it in: diff --git a/tests/ovs-xapi-sync.at b/tests/ovs-xapi-sync.at index 189c48d944dd..2f00704ba9a8 100644 --- a/tests/ovs-xapi-sync.at +++ b/tests/ovs-xapi-sync.at @@ -19,7 +19,7 @@ mkdir var var/run

Re: [ovs-dev] [PATCH 2/4] ovs-vsctl: Use default socket name in tests.

2018-02-05 Thread Yifeng Sun
It seems test "ovs-xapi-sync" is broken after this patch. I haven't figured out myself. Can you please take a look? Thanks. On Fri, Feb 2, 2018 at 1:51 PM, Ben Pfaff wrote: > By using the default socket name "db.sock", instead of "socket", we can > avoid passing --db=unix:socket

[ovs-dev] [PATCH 2/4] ovs-vsctl: Use default socket name in tests.

2018-02-02 Thread Ben Pfaff
By using the default socket name "db.sock", instead of "socket", we can avoid passing --db=unix:socket to all the ovs-vsctl invocations, which is kind of nice. Signed-off-by: Ben Pfaff --- tests/ovs-vsctl.at | 16 1 file changed, 8 insertions(+), 8 deletions(-)