diff mbox series

package/netsnmp: silence warning when running without IPv6

Message ID 20201017194135.19407-1-peter@korsgaard.com
State Accepted
Headers show
Series package/netsnmp: silence warning when running without IPv6 | expand

Commit Message

Peter Korsgaard Oct. 17, 2020, 7:41 p.m. UTC
snmpd loudly complains every 3 seconds about a failure reading
/proc/net/if_inet6 if the system does not have IPv6 support:

Jan  1 00:00:12 buildroot daemon.err snmpd[92]: ipaddress_linux: could not open /proc/net/if_inet6: No such file or directory
Jan  1 00:00:15 buildroot daemon.err snmpd[92]: ipaddress_linux: could not open /proc/net/if_inet6: No such file or directory
Jan  1 00:00:18 buildroot daemon.err snmpd[92]: ipaddress_linux: could not open /proc/net/if_inet6: No such file or directory
Jan  1 00:00:21 buildroot daemon.err snmpd[92]: ipaddress_linux: could not open /proc/net/if_inet6: No such file or directory

Add an upstream patch to only print this warning once, rather than on every
poll iteration.

Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
---
 ...once-that-opening-proc-net-if_inet6-.patch | 30 +++++++++++++++++++
 1 file changed, 30 insertions(+)
 create mode 100644 package/netsnmp/0005-MIB-II-Only-log-once-that-opening-proc-net-if_inet6-.patch

Comments

Yann E. MORIN Oct. 17, 2020, 9:57 p.m. UTC | #1
Peter, All,

On 2020-10-17 21:41 +0200, Peter Korsgaard spake thusly:
> snmpd loudly complains every 3 seconds about a failure reading
> /proc/net/if_inet6 if the system does not have IPv6 support:
> 
> Jan  1 00:00:12 buildroot daemon.err snmpd[92]: ipaddress_linux: could not open /proc/net/if_inet6: No such file or directory
> Jan  1 00:00:15 buildroot daemon.err snmpd[92]: ipaddress_linux: could not open /proc/net/if_inet6: No such file or directory
> Jan  1 00:00:18 buildroot daemon.err snmpd[92]: ipaddress_linux: could not open /proc/net/if_inet6: No such file or directory
> Jan  1 00:00:21 buildroot daemon.err snmpd[92]: ipaddress_linux: could not open /proc/net/if_inet6: No such file or directory
> 
> Add an upstream patch to only print this warning once, rather than on every
> poll iteration.
> 
> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>

Applied to masterm thanks.

Regards,
Yann E. MORIN.

> ---
>  ...once-that-opening-proc-net-if_inet6-.patch | 30 +++++++++++++++++++
>  1 file changed, 30 insertions(+)
>  create mode 100644 package/netsnmp/0005-MIB-II-Only-log-once-that-opening-proc-net-if_inet6-.patch
> 
> diff --git a/package/netsnmp/0005-MIB-II-Only-log-once-that-opening-proc-net-if_inet6-.patch b/package/netsnmp/0005-MIB-II-Only-log-once-that-opening-proc-net-if_inet6-.patch
> new file mode 100644
> index 0000000000..2d5b38eb32
> --- /dev/null
> +++ b/package/netsnmp/0005-MIB-II-Only-log-once-that-opening-proc-net-if_inet6-.patch
> @@ -0,0 +1,30 @@
> +From cd09fd82522861830aaf9d237b26eef5f9ba50d2 Mon Sep 17 00:00:00 2001
> +From: Bart Van Assche <bvanassche@acm.org>
> +Date: Wed, 21 Nov 2018 20:47:42 -0800
> +Subject: [PATCH] MIB-II: Only log once that opening /proc/net/if_inet6 failed
> +
> +If IPv6 has been disabled (ipv6.disable=1) then opening /proc/net/if_inet6
> +fails. Only log this once instead of thousand of times a day.
> +
> +Reported-by: Fif <lefif@users.sourceforge.net>
> +Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
> +---
> + agent/mibgroup/ip-mib/data_access/ipaddress_linux.c | 2 +-
> + 1 file changed, 1 insertion(+), 1 deletion(-)
> +
> +diff --git a/agent/mibgroup/ip-mib/data_access/ipaddress_linux.c b/agent/mibgroup/ip-mib/data_access/ipaddress_linux.c
> +index 5ddead3e0c..280575ce39 100644
> +--- a/agent/mibgroup/ip-mib/data_access/ipaddress_linux.c
> ++++ b/agent/mibgroup/ip-mib/data_access/ipaddress_linux.c
> +@@ -234,7 +234,7 @@ _load_v6(netsnmp_container *container, int idx_offset)
> + 
> + #define PROCFILE "/proc/net/if_inet6"
> +     if (!(in = fopen(PROCFILE, "r"))) {
> +-        snmp_log_perror("ipaddress_linux: could not open " PROCFILE);
> ++        NETSNMP_LOGONCE((LOG_ERR, "ipaddress_linux: could not open " PROCFILE));
> +         return -2;
> +     }
> + 
> +-- 
> +2.20.1
> +
> -- 
> 2.20.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
diff mbox series

Patch

diff --git a/package/netsnmp/0005-MIB-II-Only-log-once-that-opening-proc-net-if_inet6-.patch b/package/netsnmp/0005-MIB-II-Only-log-once-that-opening-proc-net-if_inet6-.patch
new file mode 100644
index 0000000000..2d5b38eb32
--- /dev/null
+++ b/package/netsnmp/0005-MIB-II-Only-log-once-that-opening-proc-net-if_inet6-.patch
@@ -0,0 +1,30 @@ 
+From cd09fd82522861830aaf9d237b26eef5f9ba50d2 Mon Sep 17 00:00:00 2001
+From: Bart Van Assche <bvanassche@acm.org>
+Date: Wed, 21 Nov 2018 20:47:42 -0800
+Subject: [PATCH] MIB-II: Only log once that opening /proc/net/if_inet6 failed
+
+If IPv6 has been disabled (ipv6.disable=1) then opening /proc/net/if_inet6
+fails. Only log this once instead of thousand of times a day.
+
+Reported-by: Fif <lefif@users.sourceforge.net>
+Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
+---
+ agent/mibgroup/ip-mib/data_access/ipaddress_linux.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/agent/mibgroup/ip-mib/data_access/ipaddress_linux.c b/agent/mibgroup/ip-mib/data_access/ipaddress_linux.c
+index 5ddead3e0c..280575ce39 100644
+--- a/agent/mibgroup/ip-mib/data_access/ipaddress_linux.c
++++ b/agent/mibgroup/ip-mib/data_access/ipaddress_linux.c
+@@ -234,7 +234,7 @@ _load_v6(netsnmp_container *container, int idx_offset)
+ 
+ #define PROCFILE "/proc/net/if_inet6"
+     if (!(in = fopen(PROCFILE, "r"))) {
+-        snmp_log_perror("ipaddress_linux: could not open " PROCFILE);
++        NETSNMP_LOGONCE((LOG_ERR, "ipaddress_linux: could not open " PROCFILE));
+         return -2;
+     }
+ 
+-- 
+2.20.1
+