Skip to content

Commit

Permalink
ofproto: Uses the VLOG_WARN_RL instead of VLOG_WARN.
Browse files Browse the repository at this point in the history
There are a lot of logs when OvS bridges, connected to controllers,
can't find the right routes. So we may use the VLOG_WARN_RL instead
of VLOG_WARN to limit the log messages. The netdev-open and
arp-lookingup are in the same case in this function.

Signed-off-by: nickcooper-zhangtonghao <nic@opencloud.tech>
Signed-off-by: Ben Pfaff <blp@ovn.org>
  • Loading branch information
xpu22 authored and blp committed Feb 6, 2017
1 parent 2504609 commit b6d7a66
Showing 1 changed file with 5 additions and 3 deletions.
8 changes: 5 additions & 3 deletions ofproto/in-band.c
Expand Up @@ -119,9 +119,11 @@ refresh_remote(struct in_band *ib, struct in_band_remote *r)
retval = netdev_get_next_hop(ib->local_netdev, &r->remote_addr.sin_addr,
&next_hop_inaddr, &next_hop_dev);
if (retval) {
VLOG_WARN("%s: cannot find route for controller ("IP_FMT"): %s",
ib->ofproto->name, IP_ARGS(r->remote_addr.sin_addr.s_addr),
ovs_strerror(retval));
VLOG_WARN_RL(&rl, "%s: cannot find route for controller "
"("IP_FMT"): %s",
ib->ofproto->name,
IP_ARGS(r->remote_addr.sin_addr.s_addr),
ovs_strerror(retval));
return 1;
}
if (!next_hop_inaddr.s_addr) {
Expand Down

0 comments on commit b6d7a66

Please sign in to comment.