On Mon, Sep 16, 2019 at 8:17 AM Aaron Conole <acon...@redhat.com> wrote:
>
> It's possible that a port added to the system with certain kinds
> of invalid parameters will cause the 'could not add' log to be
> triggered. When this happens, the vswitch run loop can continually
> re-attempt adding the port. While the parameters remain invalid
> the vswitch run loop will re-trigger the warning, flooding the
> syslog.
>
> This patch adds a simple rate limit to the log.
>
> Signed-off-by: Aaron Conole <acon...@redhat.com>
> ---
LGTM
Acked-by: William Tu <u9012...@gmail.com>
> v2: set errp
>
> vswitchd/bridge.c | 9 +++++++--
> 1 file changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c
> index d921c4ef8..8c390382f 100644
> --- a/vswitchd/bridge.c
> +++ b/vswitchd/bridge.c
> @@ -1816,8 +1816,13 @@ iface_do_create(const struct bridge *br,
> *ofp_portp = iface_pick_ofport(iface_cfg);
> error = ofproto_port_add(br->ofproto, netdev, ofp_portp);
> if (error) {
> - VLOG_WARN_BUF(errp, "could not add network device %s to ofproto
> (%s)",
> - iface_cfg->name, ovs_strerror(error));
> + static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(1, 5);
> +
> + *errp = xasprintf("could not add network device %s to ofproto (%s)",
> + iface_cfg->name, ovs_strerror(error));
> + if (!VLOG_DROP_WARN(&rl)) {
> + VLOG_WARN("%s", *errp);
> + }
> goto error;
> }
>
> --
> 2.21.0
>
> _______________________________________________
> dev mailing list
> d...@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
