[ovs-dev] [PATCH v2] tests: Avoid nonportable "sed -i".

Ben Pfaff blp at nicira.com
Wed Aug 26 22:55:04 UTC 2015

"sed -i" isn't entirely portable, and we can avoid it by using the
argument to check_logs as intended.

Signed-off-by: Ben Pfaff <blp at nicira.com>
Acked-by: Alex Wang <alexw at nicira.com>
v1->v2: Retain $1 in check_logs call from OVN_CONTROLLER_VTEP_STOP.
  Also drop other, now-unneeded, call to "sed -i".

 tests/ovn-controller-vtep.at | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/tests/ovn-controller-vtep.at b/tests/ovn-controller-vtep.at
index 9fc1526..50a7096 100644
--- a/tests/ovn-controller-vtep.at
+++ b/tests/ovn-controller-vtep.at
@@ -89,8 +89,7 @@ m4_define([OVN_CONTROLLER_VTEP_STOP],
    # sending update back to *ctl command if *ctl has not proceeded to exit yet.
    # and if *ctl command exits before database calling send, the send from
    # database will fail with 'Broken pipe' error.
-   AT_CHECK([sed -i '/Broken pipe/d' ovsdb-server.log])
-   AT_CHECK([check_logs $1])
+   AT_CHECK([check_logs $1 '/Broken pipe/d'])
    AT_CHECK([ovs-appctl -t ovs-vtep exit])
    AT_CHECK([ovs-appctl -t ovn-northd exit])
    AT_CHECK([ovs-appctl -t ovn-controller-vtep exit])
@@ -131,10 +130,6 @@ OVS_WAIT_UNTIL([test -n "`grep WARN ovn-controller-vtep.log`"])
 AT_CHECK([sed -n 's/^.*\(|WARN|.*\)$/\1/p' ovn-controller-vtep.log], [0], [dnl
 |WARN|Chassis for VTEP physical switch (br-vtep) disappears, maybe deleted by ovn-sbctl, adding it back
-# this removal of chassis could cause 'Broken pipe' warning in the ovsdb-server.log,
-# due to the race between 'ovn-sbctl' exiting and 'ovn-controller-vtep' adding
-# the chassis back.  so just removes the 'Broken pipe' warning from ovsdb-server.log.
-AT_CHECK([sed -i '/Broken pipe/d' ovsdb-server.log])
 # changes the tunnel_ip on physical switch, watches the update of chassis's
 # encap.

More information about the dev mailing list