On Sun, Jul 5, 2020 at 11:08 AM Ankur Sharma
wrote:
> From: Ankur Sharma
>
> ISSUE:
> We observed that if ovn-controller is running, while ovn-encap-ip/
> ovn-encap-type is not set, then following error gets logged
> continously:
> "chassis|INFO|Need to specify an encap type and ip"
>
> Above log increased the size of ovn-controller.log to hundreds
> of GBs before logger.d could kick in (frequency is 24 hours)
> and rotate the log file.
>
> This Patch rate limits (rate_limit=1, burst_size=5) INFO logs
> for following errors:
> a. ovn-encap-ip/ovn-encap-type not set.
> b. ovn-encap-type set to invalid value.
>
> Signed-off-by: Ankur Sharma
>
Thanks Ankur.
I applied this patch to master.
Numan
> ---
> controller/chassis.c | 6 --
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/controller/chassis.c b/controller/chassis.c
> index aa3fed0..eec270e 100644
> --- a/controller/chassis.c
> +++ b/controller/chassis.c
> @@ -214,7 +214,8 @@ chassis_parse_ovs_encap_type(const char *encap_type,
>
> SSET_FOR_EACH (type, encap_type_set) {
> if (!get_tunnel_type(type)) {
> -VLOG_INFO("Unknown tunnel type: %s", type);
> +static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(1, 5);
> +VLOG_INFO_RL(, "Unknown tunnel type: %s", type);
> }
> }
>
> @@ -266,7 +267,8 @@ chassis_parse_ovs_config(const struct
> ovsrec_open_vswitch_table *ovs_table,
> const char *encap_type = smap_get(>external_ids,
> "ovn-encap-type");
> const char *encap_ips = smap_get(>external_ids, "ovn-encap-ip");
> if (!encap_type || !encap_ips) {
> -VLOG_INFO("Need to specify an encap type and ip");
> +static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(1, 5);
> +VLOG_INFO_RL(, "Need to specify an encap type and ip");
> return false;
> }
>
> --
> 2.7.4
>
> ___
> 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
