On Tue, 3 Jul 2018 11:32:18 -0700 Ben Pfaff <[email protected]> wrote: > check_logs ignores some log messages, but it wasn't smart enough to > ignore the messages that said that the ignored messages had been > rate-limited. This fixes the problem. > > It's OK to ignore all rate-limiting messages because they only appear > if at least one message was not rate-limited, which check_logs will > catch anyway. > > Reported-by: Timothy Redaelli <[email protected]> > Reported-at: > https://mail.openvswitch.org/pipermail/ovs-discuss/2018-July/046978.html > Signed-off-by: Ben Pfaff <[email protected]> --- > tests/ofproto-macros.at | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/tests/ofproto-macros.at b/tests/ofproto-macros.at > index 7388a20a2236..2a56ae6e2f3e 100644 > --- a/tests/ofproto-macros.at > +++ b/tests/ofproto-macros.at > @@ -400,6 +400,11 @@ check_logs () { > # 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). > + # > + # We also ignore "Dropped # log messages..." messages. > Otherwise, even if > + # we ignore the messages that were rate-limited, we can end up > failing just > + # because of the announcement that rate-limiting happened (and > in a racy, > + # timing-dependent way, too). > sed -n "$1 > /reset by peer/d > /Broken pipe/d > @@ -408,6 +413,7 @@ check_logs () { > /timeval.*disk: [[0-9]]* reads, [[0-9]]* writes/d > /timeval.*context switches: [[0-9]]* voluntary, [[0-9]]* > involuntary/d /ovs_rcu.*blocked [[0-9]]* ms waiting for .* to > quiesce/d +/Dropped [[0-9]]* log messages/d > /|WARN|/p > /|ERR|/p > /|EMER|/p" ${logs}
Hi, tested on Fedora Rawhide. I still have "2018-07-04T13:54:27.012Z|00038|jsonrpc|WARN|Dropped 14 log messages in last 8 seconds (most recently, 2 seconds ago) due to excessive rate", but test doesn't fail anymore. Tested-By: Timothy Redaelli <[email protected]> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
