[ovs-git] [openvswitch/ovs] cadae7: rconn: Suppress 'connected' log for unreliable con...
GitHub
noreply at github.com
Thu Jul 5 17:16:36 UTC 2018
Branch: refs/heads/branch-2.5
Home: https://github.com/openvswitch/ovs
Commit: cadae7bc1ffbcda376ee2e12bb05b5cac3966531
https://github.com/openvswitch/ovs/commit/cadae7bc1ffbcda376ee2e12bb05b5cac3966531
Author: Ilya Maximets <i.maximets at samsung.com>
Date: 2018-07-05 (Thu, 05 Jul 2018)
Changed paths:
M lib/rconn.c
Log Message:
-----------
rconn: Suppress 'connected' log for unreliable connections.
Recent assertion failure fix changed rconn workflow for unreliable
connections (such as connections from ovs-ofctl) from
|rconn|DBG|br-int<->unix#151: entering ACTIVE
|rconn|DBG|br-int<->unix#151: connection closed by peer
|rconn|DBG|br-int<->unix#151: entering DISCONNECTED
To
|rconn|DBG|br-int<->unix#200: entering CONNECTING
|rconn|INFO|br-int<->unix#200: connected
|rconn|DBG|br-int<->unix#200: entering ACTIVE
|rconn|DBG|br-int<->unix#200: connection closed by peer
|rconn|DBG|br-int<->unix#200: entering DISCONNECTED
Many monitoring/configuring tools (ex. ovs-neutron-agent) uses
ovs-ofctl frequently to check the statuses of installed flows.
This produces a lot of "connected" logs, that are useless in general.
Fix that by changing the log level to DBG for unreliable connections.
Suggested-by: Ben Pfaff <blp at ovn.org>
Fixes: c9a9b9b00bf5 ("rconn: Introduce new invariant to fix assertion failure in corner case.")
Signed-off-by: Ilya Maximets <i.maximets at samsung.com>
Signed-off-by: Ben Pfaff <blp at ovn.org>
**NOTE:** This service been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/
Functionality will be removed from GitHub.com on January 31st, 2019.
More information about the git
mailing list