Thanks Alex, I applied this to master.
On Thu, Aug 27, 2015 at 06:21:03PM -0700, Alex Wang wrote:
> Looks good to me,
>
> Thx for fixing it~
>
> On Thu, Aug 27, 2015 at 11:11 AM, Ben Pfaff <b...@nicira.com> wrote:
>
> > "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 <b...@nicira.com>
> > Acked-by: Alex Wang <al...@nicira.com>
> > ---
> > v1->v2: Retain $1 in check_logs call from OVN_CONTROLLER_VTEP_STOP.
> > Also drop other, now-unneeded, call to "sed -i".
> > v2->v3: Merge multiple check_logs arguments into one, since it only
> > supports one.
> >
> > tests/ovn-controller-vtep.at | 14 +++++---------
> > 1 file changed, 5 insertions(+), 9 deletions(-)
> >
> > diff --git a/tests/ovn-controller-vtep.at b/tests/ovn-controller-vtep.at
> > index 9fc1526..063feeb 100644
> > --- a/tests/ovn-controller-vtep.at
> > +++ b/tests/ovn-controller-vtep.at
> > @@ -89,8 +89,8 @@ 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 +131,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.
> > @@ -179,7 +175,7 @@ AT_CHECK([ovn-sbctl --columns=vtep_logical_switches
> > list Chassis | cut -d ':' -f
> > [[]]
> > ])
> >
> > -OVN_CONTROLLER_VTEP_STOP(["/Chassis for VTEP physical switch (br-vtep)
> > disappears/d"])
> > +OVN_CONTROLLER_VTEP_STOP([/Chassis for VTEP physical switch (br-vtep)
> > disappears/d])
> > AT_CLEANUP
> >
> >
> > @@ -242,7 +238,7 @@ AT_CHECK_UNQUOTED([ovn-sbctl --columns=chassis list
> > Port_Binding | cut -d ':' -f
> > [[]]
> > ])
> >
> > -OVN_CONTROLLER_VTEP_STOP(["/has already been associated with logical
> > port/d"])
> > +OVN_CONTROLLER_VTEP_STOP([/has already been associated with logical
> > port/d])
> > AT_CLEANUP
> >
> >
> > @@ -282,5 +278,5 @@ AT_CHECK_UNQUOTED([ovn-sbctl --columns=chassis list
> > Port_Binding br-vtep_lswitch
> > ${chassis_uuid}
> > ])
> >
> > -OVN_CONTROLLER_VTEP_STOP(["/has already been associated with logical
> > datapath/d"])
> > +OVN_CONTROLLER_VTEP_STOP([/has already been associated with logical
> > datapath/d])
> > AT_CLEANUP
> > --
> > 2.1.3
> >
> >
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
