Skip to content

Commit

Permalink
ofproto-macros.at: Ignore "Socket is not connected" log messages.
Browse files Browse the repository at this point in the history
FreeBSD likely reports ENOTCONN instead of EPIPE/ECONNRESET in case
of sending to the disconnected socket.

Signed-off-by: Ilya Maximets <i.maximets@samsung.com>
Signed-off-by: Ben Pfaff <blp@ovn.org>
  • Loading branch information
igsilya authored and blp committed Dec 27, 2018
1 parent fe495d0 commit cb334ff
Showing 1 changed file with 6 additions and 4 deletions.
10 changes: 6 additions & 4 deletions tests/ofproto-macros.at
Expand Up @@ -396,10 +396,11 @@ check_logs () {
# database. However, in reaction, some daemon may immediately update the
# database, and this later update may cause database sending update back to
# *ctl command if *ctl has not exited yet. If *ctl command exits before
# the database calls send, the send fails with 'Broken pipe'. Also removes
# all "connection reset" warning logs for similar reasons (either EPIPE or
# ECONNRESET can be returned on a send depending on whether the peer had
# unconsumed data when it closed the socket).
# the database calls send, the send fails with 'Broken pipe' or
# 'not connected' depending on system. Also removes all 'connection reset'
# warning logs for similar reasons (EPIPE, ENOTCONN or ECONNRESET can be
# returned on a send depending on whether the peer had unconsumed data
# when it closed the socket).
#
# We also ignore "Dropped # log messages..." messages. Otherwise, even if
# we ignore the messages that were rate-limited, we can end up failing just
Expand All @@ -408,6 +409,7 @@ check_logs () {
sed -n "$1
/reset by peer/d
/Broken pipe/d
/is not connected/d
/timeval.*Unreasonably long [[0-9]]*ms poll interval/d
/timeval.*faults: [[0-9]]* minor, [[0-9]]* major/d
/timeval.*disk: [[0-9]]* reads, [[0-9]]* writes/d
Expand Down

0 comments on commit cb334ff

Please sign in to comment.