Thanks. Tested-by: Yifeng Sun <[email protected]>
Reviewed-by: Yifeng Sun <[email protected]> On Tue, Feb 6, 2018 at 10:15 AM, Ben Pfaff <[email protected]> wrote: > 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 > touch var/run/xapi_init_complete.cookie > > ovs_vsctl () { > - ovs-vsctl --no-wait -vreconnect:emer --db=unix:socket "$@" > + ovs-vsctl --no-wait -vreconnect:emer "$@" > } > > # Start ovsdb-server. > @@ -27,7 +27,7 @@ OVS_VSCTL_SETUP > > # Start ovs-xapi-sync. > AT_CHECK([$PYTHON ./ovs-xapi-sync "--pidfile=ovs-xapi-sync.pid" \ > - "--root-prefix=`pwd`" unix:socket >log 2>&1 &]) > + "--root-prefix=`pwd`" unix:db.sock >log 2>&1 &]) > AT_CAPTURE_FILE([log]) > > # Add bridges and check ovs-xapi-sync's work. > > > On Mon, Feb 05, 2018 at 02:13:21PM -0800, Yifeng Sun wrote: > > 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 <[email protected]> wrote: > > > > > 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 <[email protected]> > > > --- > > > tests/ovs-vsctl.at | 16 ++++++++-------- > > > 1 file changed, 8 insertions(+), 8 deletions(-) > > > > > > diff --git a/tests/ovs-vsctl.at b/tests/ovs-vsctl.at > > > index 2c4c45d70947..415e14833249 100644 > > > --- a/tests/ovs-vsctl.at > > > +++ b/tests/ovs-vsctl.at > > > @@ -4,7 +4,7 @@ dnl Creates an empty database in the current directory > and > > > then starts > > > dnl an ovsdb-server on it for ovs-vsctl to connect to. > > > m4_define([OVS_VSCTL_SETUP], > > > [OVSDB_INIT([db]) > > > - AT_CHECK([ovsdb-server --detach --no-chdir --pidfile > > > --remote=punix:socket db >/dev/null 2>&1], [0], [ignore], [ignore]) > > > + AT_CHECK([ovsdb-server --detach --no-chdir --pidfile > > > --remote=punix:db.sock db >/dev/null 2>&1], [0], [ignore], [ignore]) > > > on_exit 'kill `cat ovsdb-server.pid`']) > > > > > > dnl OVS_VSCTL_CLEANUP > > > @@ -16,17 +16,17 @@ dnl RUN_OVS_VSCTL(COMMAND, ...) > > > dnl > > > dnl Executes each ovs-vsctl COMMAND. > > > m4_define([RUN_OVS_VSCTL], > > > - [m4_foreach([command], [$@], [ovs-vsctl --no-wait -vreconnect:emer > > > --db=unix:socket command > > > + [m4_foreach([command], [$@], [ovs-vsctl --no-wait -vreconnect:emer > > > command > > > ])]) > > > m4_define([RUN_OVS_VSCTL_ONELINE], > > > - [m4_foreach([command], [$@], [ovs-vsctl --no-wait -vreconnect:emer > > > --db=unix:socket --oneline -- command > > > + [m4_foreach([command], [$@], [ovs-vsctl --no-wait -vreconnect:emer > > > --oneline -- command > > > ])]) > > > > > > dnl RUN_OVS_VSCTL_TOGETHER(COMMAND, ...) > > > dnl > > > dnl Executes each ovs-vsctl COMMAND in a single run of ovs-vsctl. > > > m4_define([RUN_OVS_VSCTL_TOGETHER], > > > - [ovs-vsctl --no-wait -vreconnect:emer --db=unix:socket --oneline dnl > > > + [ovs-vsctl --no-wait -vreconnect:emer --oneline dnl > > > m4_foreach([command], [$@], [ -- command])]) > > > > > > dnl CHECK_BRIDGES([BRIDGE, PARENT, VLAN], ...) > > > @@ -875,7 +875,7 @@ AT_CHECK( > > > > > > ]) > > > m4_define([VSCTL_CHECK_FIND], > > > - [AT_CHECK([echo `ovs-vsctl --bare --no-wait -vreconnect:emer > > > --db=unix:socket -- --columns=name find bridge '$1' | sort`], [0], [$2 > > > + [AT_CHECK([echo `ovs-vsctl --bare --no-wait -vreconnect:emer -- > > > --columns=name find bridge '$1' | sort`], [0], [$2 > > > ])]) > > > > > > # Arithmetic relational operators without keys. > > > @@ -1078,19 +1078,19 @@ AT_SETUP([unreferenced record warnings]) > > > AT_KEYWORDS([ovs-vsctl]) > > > OVS_VSCTL_SETUP > > > AT_CHECK( > > > - [ovs-vsctl -vPATTERN:console:'%c|%p|%m' --no-wait -vreconnect:emer > > > --db=unix:socket \ > > > + [ovs-vsctl -vPATTERN:console:'%c|%p|%m' --no-wait -vreconnect:emer \ > > > -- create Bridge name=br0 | uuidfilt], > > > [0], [<0> > > > ], [db_ctl_base|WARN|applying "create" command to table Bridge without > > > --id option will have no effect > > > ]) > > > AT_CHECK( > > > - [ovs-vsctl -vPATTERN:console:'%c|%p|%m' --no-wait -vreconnect:emer > > > --db=unix:socket \ > > > + [ovs-vsctl -vPATTERN:console:'%c|%p|%m' --no-wait -vreconnect:emer \ > > > -- --id=@br0 create Bridge name=br0 | uuidfilt], > > > [0], [<0> > > > ], [vsctl|WARN|row id "@br0" was created but no reference to it was > > > inserted, so it will not actually appear in the database > > > ]) > > > AT_CHECK( > > > - [ovs-vsctl -vPATTERN:console:'%c|%p|%m' --no-wait -vreconnect:emer > > > --db=unix:socket \ > > > + [ovs-vsctl -vPATTERN:console:'%c|%p|%m' --no-wait -vreconnect:emer \ > > > -- --id=@eth0_iface create Interface name=eth0 \ > > > -- --id=@eth0 create Port name=eth0 interfaces=@eth0_iface \ > > > -- --id=@m0 create Mirror name=m0 output_port=@eth0 \ > > > -- > > > 2.15.1 > > > > > > _______________________________________________ > > > dev mailing list > > > [email protected] > > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > > > > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
