Thx pushed this series to master,~ On Sun, Aug 16, 2015 at 2:15 PM, Russell Bryant <rbry...@redhat.com> wrote:
> On 08/16/2015 12:56 PM, Alex Wang wrote: > > When testing the recreation of 'chassis' table entry by 'ovn-controller- > > vtep'. The removal of 'chassis' table entry by the 'ovn-sbctl' could > > cause 'Broken pipe' warning in ovsdb-server.log. This is due to the > > race between 'ovn-sbctl' exiting and 'ovn-controller-vtep' adding > > the chassis back. So, if the 'ovn-sbctl' exits right when the > > ovsdb-server tries to send update of readd of the deleted 'chassis', > > the sending will fail with 'Broken pipe' error. > > > > Signed-off-by: Alex Wang <al...@nicira.com> > > --- > > tests/ovn-controller-vtep.at | 4 ++++ > > 1 file changed, 4 insertions(+) > > > > diff --git a/tests/ovn-controller-vtep.at b/tests/ovn-controller-vtep.at > > index 82f6166..12d0134 100644 > > --- a/tests/ovn-controller-vtep.at > > +++ b/tests/ovn-controller-vtep.at > > @@ -104,6 +104,10 @@ 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. > > > > Acked-by: Russell Bryant <rbry...@redhat.com> > > -- > Russell Bryant > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
