[regression] SNMP missing disks in hrStorageTable

Bug #1843036 reported by Eric Desrochers on 2019-09-06
154
This bug affects 29 people
Affects Status Importance Assigned to Milestone
net-snmp (Ubuntu)
High
Eric Desrochers
Xenial
High
Eric Desrochers
Bionic
High
Eric Desrochers
Disco
High
Eric Desrochers
Eoan
High
Eric Desrochers

Bug Description

[IMPACT]

It has been brought to me the following:

Some hosts have started to cause UNKNOWN return values in Nagios for checks on their disks. This is because these hosts are no longer reporting their disks as part of the SNMP table hrStorageTable (1.3.6.1.2.1.25.2.3.1 ) - only memory devices are being reported. The affected hosts that I have investigated received updates for SNMP:

Upgrade package libsnmp-base 5.7.3+dfsg-1.8ubuntu3.1 to 5.7.3+dfsg-1.8ubuntu3.2
Upgrade package libsnmp30 5.7.3+dfsg-1.8ubuntu3.1 to 5.7.3+dfsg-1.8ubuntu3.2
Upgrade package snmpd 5.7.3+dfsg-1.8ubuntu3.1 to 5.7.3+dfsg-1.8ubuntu3.2

It seems likely that this package update is the cause.

As debug info, you can see the difference between 2 nearly identical servers, one of which received the SNMP updates, and one which did not. You can see that the one without the update is returning disks in the SNMP output:

# snmpwalk -v2c -cpublic arcprsmt01 1.3.6.1.2.1.25.2.3.1.3
iso.3.6.1.2.1.25.2.3.1.3.1 = STRING: "Physical memory"
iso.3.6.1.2.1.25.2.3.1.3.3 = STRING: "Virtual memory"
iso.3.6.1.2.1.25.2.3.1.3.6 = STRING: "Memory buffers"
iso.3.6.1.2.1.25.2.3.1.3.7 = STRING: "Cached memory"
iso.3.6.1.2.1.25.2.3.1.3.8 = STRING: "Shared memory"
iso.3.6.1.2.1.25.2.3.1.3.10 = STRING: "Swap space"
iso.3.6.1.2.1.25.2.3.1.3.31 = STRING: "/"
iso.3.6.1.2.1.25.2.3.1.3.37 = STRING: "/run"
iso.3.6.1.2.1.25.2.3.1.3.39 = STRING: "/dev/shm"
iso.3.6.1.2.1.25.2.3.1.3.40 = STRING: "/run/lock"
iso.3.6.1.2.1.25.2.3.1.3.41 = STRING: "/sys/fs/cgroup"
iso.3.6.1.2.1.25.2.3.1.3.67 = STRING: "/run/snapd/ns"
iso.3.6.1.2.1.25.2.3.1.3.70 = STRING: "/var/lib/docker/containers/3cad3d36991b677c37b08b374a7bfeceddf36a6b6754edaa1ff687b00111a6b8/mounts/shm"
iso.3.6.1.2.1.25.2.3.1.3.73 = STRING: "/var/lib/docker/containers/c605c4b76dea65d562ba024212a38e24fb710186c499187b6604478b7ff678e9/mounts/shm"
iso.3.6.1.2.1.25.2.3.1.3.82 = STRING: "/run/user/2002"
iso.3.6.1.2.1.25.2.3.1.3.253 = STRING: "/var/lib/docker/containers/dc74a157fbaaa284e0e5b8ca4afc88769bf625eb796d89a5d26f98a540cabf35/mounts/shm"
iso.3.6.1.2.1.25.2.3.1.3.256 = STRING: "/var/lib/docker/containers/6ce6193433f9c1c95cccbfbbe08a3f3385bdbc4f2e3f0baa02d11baf3866dfd2/mounts/shm"
iso.3.6.1.2.1.25.2.3.1.3.258 = STRING: "/run/user/1000"

The other, which received SNMP updates, is returning only memory devices, such as swap and shmem:

# snmpwalk -v2c -cpublic arcprsmt02 1.3.6.1.2.1.25.2.3.1.3
iso.3.6.1.2.1.25.2.3.1.3.1 = STRING: "Physical memory"
iso.3.6.1.2.1.25.2.3.1.3.3 = STRING: "Virtual memory"
iso.3.6.1.2.1.25.2.3.1.3.6 = STRING: "Memory buffers"
iso.3.6.1.2.1.25.2.3.1.3.7 = STRING: "Cached memory"
iso.3.6.1.2.1.25.2.3.1.3.8 = STRING: "Shared memory"
iso.3.6.1.2.1.25.2.3.1.3.10 = STRING: "Swap space"

[Test Case]

* Install snmp snmpd
* Configure /etc/snmp/snmpd.conf by adding the following:
 view systemonly included .1.3.6.1.2.1.25.2.3.1.3
* Restart snmpd
* Use snmpwalk:
 ** snmpwalk -v2c -cpublic localhost 1.3.6.1.2.1.25.2.3.1.3

Expected behavior is to see the disk as follow:
"
iso.3.6.1.2.1.25.2.3.1.3.1 = STRING: "Physical memory"
iso.3.6.1.2.1.25.2.3.1.3.3 = STRING: "Virtual memory"
iso.3.6.1.2.1.25.2.3.1.3.6 = STRING: "Memory buffers"
iso.3.6.1.2.1.25.2.3.1.3.7 = STRING: "Cached memory"
iso.3.6.1.2.1.25.2.3.1.3.8 = STRING: "Shared memory"
iso.3.6.1.2.1.25.2.3.1.3.10 = STRING: "Swap space"
iso.3.6.1.2.1.25.2.3.1.3.31 = STRING: "/"
iso.3.6.1.2.1.25.2.3.1.3.33 = STRING: "/dev"
iso.3.6.1.2.1.25.2.3.1.3.45 = STRING: "/dev/lxd"
iso.3.6.1.2.1.25.2.3.1.3.46 = STRING: "/dev/.lxd-mounts"
iso.3.6.1.2.1.25.2.3.1.3.63 = STRING: "/proc/sys/kernel/random/boot_id"
iso.3.6.1.2.1.25.2.3.1.3.66 = STRING: "/dev/shm"
iso.3.6.1.2.1.25.2.3.1.3.67 = STRING: "/run"
iso.3.6.1.2.1.25.2.3.1.3.68 = STRING: "/run/lock"
iso.3.6.1.2.1.25.2.3.1.3.69 = STRING: "/sys/fs/cgroup"
"

[Potential Regression]
The fix has been tested by various impacted users (+/- 10 different impacted users), and feedback were all positives (see comments below).

Note that this fix a regression introduced by: https://bugs.launchpad.net/bugs/1835818

[Other information]

# Upstream commit:
https://github.com/net-snmp/net-snmp/commit/71e487212bd65839e7454df9701524d08cf0d74f
https://github.com/net-snmp/net-snmp/commit/bcb1a6b8afc444bbcd099a195e08f0b01cbc8f6b

# git describe --contains 71e487212bd65839e7454df9701524d08cf0d74f
v5.8.pre1

# rmadison:
 net-snmp | 5.7.3+dfsg-1ubuntu4 | xenial | source
 net-snmp | 5.7.3+dfsg-1ubuntu4.2 | xenial-security | source
 net-snmp | 5.7.3+dfsg-1ubuntu4.2 | xenial-updates | source
 net-snmp | 5.7.3+dfsg-1ubuntu4.3 | xenial-proposed | source
 net-snmp | 5.7.3+dfsg-1.8ubuntu3 | bionic | source
 net-snmp | 5.7.3+dfsg-1.8ubuntu3.1 | bionic-security | source
 net-snmp | 5.7.3+dfsg-1.8ubuntu3.1 | bionic-updates | source
 net-snmp | 5.7.3+dfsg-1.8ubuntu3.2 | bionic-proposed | source
 net-snmp | 5.7.3+dfsg-5ubuntu1 | disco | source
 net-snmp | 5.7.3+dfsg-5ubuntu1.1 | disco-proposed | source
 net-snmp | 5.7.3+dfsg-5ubuntu3 | eoan | source

Eric Desrochers (slashd) on 2019-09-06
tags: added: regression-update sts
Eric Desrochers (slashd) wrote :

Others started to report the regression as well:
https://sourceforge.net/p/net-snmp/patches/1350/

Changed in net-snmp (Ubuntu):
importance: Undecided → High
Eric Desrochers (slashd) wrote :

Look like a regression introduced by : https://bugs.launchpad.net/bugs/1835818

Eric Desrochers (slashd) on 2019-09-06
Changed in net-snmp (Ubuntu Bionic):
importance: Undecided → High
status: New → In Progress
assignee: nobody → Eric Desrochers (slashd)
Changed in net-snmp (Ubuntu Eoan):
assignee: nobody → Eric Desrochers (slashd)
Changed in net-snmp (Ubuntu Disco):
assignee: nobody → Eric Desrochers (slashd)
Changed in net-snmp (Ubuntu Xenial):
assignee: nobody → Eric Desrochers (slashd)
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in net-snmp (Ubuntu Disco):
status: New → Confirmed
Changed in net-snmp (Ubuntu Xenial):
status: New → Confirmed
Changed in net-snmp (Ubuntu):
status: New → Confirmed
Joao Silva (joaomas) wrote :

We have a similar problem with the Nagios:

(...)
   OID : 1.3.6.1.2.1.25.2.3.1.2.1, Storagetype: Ram ?= FixedDisk
OID : 1.3.6.1.2.1.25.2.3.1.3.7, Desc : Cached memory
   OID : 1.3.6.1.2.1.25.2.3.1.2.7, Storagetype: Other ?= FixedDisk
storages selected : 0
Unknown storage : /home$ : ERROR

After downgrade:
(..)
OID : 1.3.6.1.2.1.25.2.3.1.3.40, Desc : /run/lock
   OID : 1.3.6.1.2.1.25.2.3.1.2.40, Storagetype: FixedDisk ?= FixedDisk
   Name : /run/lock, Index : 40
OID : 1.3.6.1.2.1.25.2.3.1.3.73, Desc : /home/xxxx
(...)
All selected storages (<90%) : OK

Joseph Gooch (whitemystic) wrote :

Need this too if you haven't included it...

Eric Desrochers (slashd) wrote :

Thanks Joseph for your contribution on this, good catch on the missing ().

I will build a new test package and will make it available in the next hour.

Joseph Gooch (whitemystic) wrote :

Docker test case attached - if you untar this, and
cd DockerTestCase
docker build .

It'll run through, download the pkg from your PPA and illustrate that OIDs above 30 are missing.

8ubuntu3.1 returns this:
iso.3.6.1.2.1.25.2.3.1.3.1 = STRING: "Physical memory"
iso.3.6.1.2.1.25.2.3.1.3.3 = STRING: "Virtual memory"
iso.3.6.1.2.1.25.2.3.1.3.6 = STRING: "Memory buffers"
iso.3.6.1.2.1.25.2.3.1.3.7 = STRING: "Cached memory"
iso.3.6.1.2.1.25.2.3.1.3.8 = STRING: "Shared memory"
iso.3.6.1.2.1.25.2.3.1.3.10 = STRING: "Swap space"
iso.3.6.1.2.1.25.2.3.1.3.34 = STRING: "/dev"
iso.3.6.1.2.1.25.2.3.1.3.37 = STRING: "/sys/fs/cgroup"
iso.3.6.1.2.1.25.2.3.1.3.51 = STRING: "/dev/shm"
iso.3.6.1.2.1.25.2.3.1.3.52 = STRING: "/etc/resolv.conf"
iso.3.6.1.2.1.25.2.3.1.3.53 = STRING: "/etc/hostname"
iso.3.6.1.2.1.25.2.3.1.3.54 = STRING: "/etc/hosts"
iso.3.6.1.2.1.25.2.3.1.3.60 = STRING: "/proc/acpi"
iso.3.6.1.2.1.25.2.3.1.3.61 = STRING: "/proc/kcore"
iso.3.6.1.2.1.25.2.3.1.3.62 = STRING: "/proc/keys"
iso.3.6.1.2.1.25.2.3.1.3.63 = STRING: "/proc/timer_list"
iso.3.6.1.2.1.25.2.3.1.3.64 = STRING: "/proc/sched_debug"
iso.3.6.1.2.1.25.2.3.1.3.65 = STRING: "/proc/scsi"
iso.3.6.1.2.1.25.2.3.1.3.66 = STRING: "/sys/firmware"

PPA:
iso.3.6.1.2.1.25.2.3.1.3.1 = STRING: "Physical memory"
iso.3.6.1.2.1.25.2.3.1.3.3 = STRING: "Virtual memory"
iso.3.6.1.2.1.25.2.3.1.3.6 = STRING: "Memory buffers"
iso.3.6.1.2.1.25.2.3.1.3.7 = STRING: "Cached memory"
iso.3.6.1.2.1.25.2.3.1.3.8 = STRING: "Shared memory"
iso.3.6.1.2.1.25.2.3.1.3.10 = STRING: "Swap space"

I did see you'll do another package, so when that's posted I'll just run this again. :)

Eric Desrochers (slashd) wrote :

It is already upstream'ed via:

git show 71e487212b
commit 71e487212bd65839e7454df9701524d08cf0d74f
Author: Niels Baggesen <email address hidden>
Date: Thu Jul 21 21:45:26 2016 +0200

    fsys.h: put paranthesis around macros which are expressions

diff --git a/include/net-snmp/agent/hardware/fsys.h b/include/net-snmp/agent/hardware/fsys.h
index cd7880ce7..b540a53d4 100644
--- a/include/net-snmp/agent/hardware/fsys.h
+++ b/include/net-snmp/agent/hardware/fsys.h
@@ -33,14 +33,14 @@ typedef struct netsnmp_fsys_info_s netsnmp_fsys_info;
    /*
     * Additional enumerationis - not listed in that MIB
     */
-#define NETSNMP_FS_TYPE_IGNORE 1 | _NETSNMP_FS_TYPE_LOCAL | _NETSNMP_FS_TYPE_SKIP_BIT
+#define NETSNMP_FS_TYPE_IGNORE (1 | _NETSNMP_FS_TYPE_LOCAL | _NETSNMP_FS_TYPE_SKIP_BIT)

-#define NETSNMP_FS_TYPE_PROC 2 | _NETSNMP_FS_TYPE_LOCAL | _NETSNMP_FS_TYPE_SKIP_BIT
+#define NETSNMP_FS_TYPE_PROC (2 | _NETSNMP_FS_TYPE_LOCAL | _NETSNMP_FS_TYPE_SKIP_BIT)

-#define NETSNMP_FS_TYPE_DEVPTS 3 | _NETSNMP_FS_TYPE_LOCAL | _NETSNMP_FS_TYPE_SKIP_BIT
-#define NETSNMP_FS_TYPE_SYSFS 4 | _NETSNMP_FS_TYPE_LOCAL | _NETSNMP_FS_TYPE_SKIP_BIT
-#define NETSNMP_FS_TYPE_TMPFS 5 | _NETSNMP_FS_TYPE_LOCAL
-#define NETSNMP_FS_TYPE_USBFS 6 | _NETSNMP_FS_TYPE_LOCAL
+#define NETSNMP_FS_TYPE_DEVPTS (3 | _NETSNMP_FS_TYPE_LOCAL | _NETSNMP_FS_TYPE_SKIP_BIT)
+#define NETSNMP_FS_TYPE_SYSFS (4 | _NETSNMP_FS_TYPE_LOCAL | _NETSNMP_FS_TYPE_SKIP_BIT)
+#define NETSNMP_FS_TYPE_TMPFS (5 | _NETSNMP_FS_TYPE_LOCAL)
+#define NETSNMP_FS_TYPE_USBFS (6 | _NETSNMP_FS_TYPE_LOCAL)

 #define NETSNMP_FS_FLAG_ACTIVE 0x01
 #define NETSNMP_FS_FLAG_REMOTE 0x02

Only thing that need to be address upstream is the useless preprocessor HAVE_GETFSSTAT directive.

Eric Desrochers (slashd) on 2019-09-06
Changed in net-snmp (Ubuntu Eoan):
status: Confirmed → In Progress
Changed in net-snmp (Ubuntu Disco):
status: Confirmed → In Progress
Changed in net-snmp (Ubuntu Xenial):
status: Confirmed → In Progress
Changed in net-snmp (Ubuntu Disco):
importance: Undecided → High
Changed in net-snmp (Ubuntu Xenial):
importance: Undecided → High
Eric Desrochers (slashd) wrote :

Here we go:

sudo add-apt-repository ppa:slashd/lp1843036
sudo apt-get update

Version: 5.7.3+dfsg-1.8ubuntu3.2+testpkg20190906b3

If it works as expected, I'll upload the change over the weekend in E and then will star the SRU for D/B/X.

Eric Desrochers (slashd) wrote :

Look good for me using "5.7.3+dfsg-1.8ubuntu3.2+testpkg20190906b3"

# dpkg -l | grep -i snmp
ii libsnmp-base 5.7.3+dfsg-1.8ubuntu3.2+testpkg20190906b3 all SNMP configuration script, MIBs and documentation
ii libsnmp30:amd64 5.7.3+dfsg-1.8ubuntu3.2+testpkg20190906b3 amd64 SNMP (Simple Network Management Protocol) library
ii snmp 5.7.3+dfsg-1.8ubuntu3.2+testpkg20190906b3 amd64 SNMP (Simple Network Management Protocol) applications
ii snmpd 5.7.3+dfsg-1.8ubuntu3.2+testpkg20190906b3 amd64 SNMP (Simple Network Management Protocol) agents

# snmpwalk -v2c -cpublic localhost 1.3.6.1.2.1.25.2.3.1.3
iso.3.6.1.2.1.25.2.3.1.3.1 = STRING: "Physical memory"
iso.3.6.1.2.1.25.2.3.1.3.3 = STRING: "Virtual memory"
iso.3.6.1.2.1.25.2.3.1.3.6 = STRING: "Memory buffers"
iso.3.6.1.2.1.25.2.3.1.3.7 = STRING: "Cached memory"
iso.3.6.1.2.1.25.2.3.1.3.8 = STRING: "Shared memory"
iso.3.6.1.2.1.25.2.3.1.3.10 = STRING: "Swap space"
iso.3.6.1.2.1.25.2.3.1.3.31 = STRING: "/"
iso.3.6.1.2.1.25.2.3.1.3.33 = STRING: "/dev"
iso.3.6.1.2.1.25.2.3.1.3.45 = STRING: "/dev/lxd"
iso.3.6.1.2.1.25.2.3.1.3.46 = STRING: "/dev/.lxd-mounts"
iso.3.6.1.2.1.25.2.3.1.3.63 = STRING: "/proc/sys/kernel/random/boot_id"
iso.3.6.1.2.1.25.2.3.1.3.66 = STRING: "/dev/shm"
iso.3.6.1.2.1.25.2.3.1.3.67 = STRING: "/run"
iso.3.6.1.2.1.25.2.3.1.3.68 = STRING: "/run/lock"
iso.3.6.1.2.1.25.2.3.1.3.69 = STRING: "/sys/fs/cgroup"
iso.3.6.1.2.1.25.2.3.1.3.69 = No more variables left in this MIB View (It is past the end of the MIB tree)

Chris Newcomer (cnewcomer) wrote :

I also tested the test package, 5.7.3+dfsg-1.8ubuntu3.2+testpkg20190906b3, in a direct-mapped autofs environment that was the basis for the fix that caused this regression.

The test package does fix the original autofs "mount everything" issue and allows for snmpwalk to monitor the mounted disks as well.

I even tested it with a single autofs directory mounted and it worked with that as well.

root@test1:~# df
Filesystem 1K-blocks Used Available Use% Mounted on
udev 989468 0 989468 0% /dev
tmpfs 204136 672 203464 1% /run
/dev/vda1 10254592 3972248 5742084 41% /
tmpfs 1020664 0 1020664 0% /dev/shm
tmpfs 5120 0 5120 0% /run/lock
tmpfs 1020664 0 1020664 0% /sys/fs/cgroup
tmpfs 204132 0 204132 0% /run/user/1000
root@test1:~# patch /etc/snmp/snmpd.conf < /var/tmp/snmpd.conf.patch
patching file /etc/snmp/snmpd.conf
root@test1:~# systemctl restart snmpd
root@test1:~# snmpwalk -v2c -cpublic localhost 1.3.6.1.2.1.25.2.3.1.3
iso.3.6.1.2.1.25.2.3.1.3.1 = STRING: "Physical memory"
iso.3.6.1.2.1.25.2.3.1.3.3 = STRING: "Virtual memory"
iso.3.6.1.2.1.25.2.3.1.3.6 = STRING: "Memory buffers"
iso.3.6.1.2.1.25.2.3.1.3.7 = STRING: "Cached memory"
iso.3.6.1.2.1.25.2.3.1.3.8 = STRING: "Shared memory"
iso.3.6.1.2.1.25.2.3.1.3.10 = STRING: "Swap space"
iso.3.6.1.2.1.25.2.3.1.3.31 = STRING: "/"
iso.3.6.1.2.1.25.2.3.1.3.37 = STRING: "/run"
iso.3.6.1.2.1.25.2.3.1.3.39 = STRING: "/dev/shm"
iso.3.6.1.2.1.25.2.3.1.3.40 = STRING: "/run/lock"
iso.3.6.1.2.1.25.2.3.1.3.41 = STRING: "/sys/fs/cgroup"
iso.3.6.1.2.1.25.2.3.1.3.63 = STRING: "/run/user/1000"
iso.3.6.1.2.1.25.2.3.1.3.63 = No more variables left in this MIB View (It is past the end of the MIB tree)
root@test1:~# dpkg -l | grep snmpd
ii snmpd 5.7.3+dfsg-1.8ubuntu3.2+testpkg20190906b3 amd64 SNMP (Simple Network Management Protocol) agents

### WITH 1 AUTOFS DIRECTORY MOUNTED: /media/test100 ###
root@test1:~# ls /media/test100
root@test1:~# snmpwalk -v2c -cpublic localhost 1.3.6.1.2.1.25.2.3.1.3
iso.3.6.1.2.1.25.2.3.1.3.1 = STRING: "Physical memory"
iso.3.6.1.2.1.25.2.3.1.3.3 = STRING: "Virtual memory"
iso.3.6.1.2.1.25.2.3.1.3.6 = STRING: "Memory buffers"
iso.3.6.1.2.1.25.2.3.1.3.7 = STRING: "Cached memory"
iso.3.6.1.2.1.25.2.3.1.3.8 = STRING: "Shared memory"
iso.3.6.1.2.1.25.2.3.1.3.10 = STRING: "Swap space"
iso.3.6.1.2.1.25.2.3.1.3.31 = STRING: "/"
iso.3.6.1.2.1.25.2.3.1.3.37 = STRING: "/run"
iso.3.6.1.2.1.25.2.3.1.3.39 = STRING: "/dev/shm"
iso.3.6.1.2.1.25.2.3.1.3.40 = STRING: "/run/lock"
iso.3.6.1.2.1.25.2.3.1.3.41 = STRING: "/sys/fs/cgroup"
iso.3.6.1.2.1.25.2.3.1.3.63 = STRING: "/run/user/1000"
iso.3.6.1.2.1.25.2.3.1.3.162 = STRING: "/media/test100"
iso.3.6.1.2.1.25.2.3.1.3.162 = No more variables left in this MIB View (It is past the end of the MIB tree)

Joseph Gooch (whitemystic) wrote :

Confirmed testpkg20190906b3 fixes the issue for me as well.

I imagine there will need to be a xenial backport too?

Eric Desrochers (slashd) on 2019-09-06
description: updated
description: updated
Eric Desrochers (slashd) wrote :

Uploaded in active devel release "Eoan". Will do the SRU (X/B/D) once it lands in -release.

- Eric

Changed in net-snmp (Ubuntu Eoan):
status: In Progress → Fix Committed
Eric Desrochers (slashd) on 2019-09-06
description: updated
Eric Desrochers (slashd) wrote :

[VERIFICATION EOAN]

# dpkg -l | grep -i snmp
ii libsnmp-base 5.7.3+dfsg-5ubuntu4 all SNMP configuration script, MIBs and documentation
ii libsnmp30:amd64 5.7.3+dfsg-5ubuntu4 amd64 SNMP (Simple Network Management Protocol) library
ii snmp 5.7.3+dfsg-5ubuntu4 amd64 SNMP (Simple Network Management Protocol) applications
ii snmpd 5.7.3+dfsg-5ubuntu4 amd64 SNMP (Simple Network Management Protocol) agents

# snmpwalk -v2c -cpublic localhost 1.3.6.1.2.1.25.2.3.1.3
iso.3.6.1.2.1.25.2.3.1.3.1 = STRING: "Physical memory"
iso.3.6.1.2.1.25.2.3.1.3.3 = STRING: "Virtual memory"
iso.3.6.1.2.1.25.2.3.1.3.6 = STRING: "Memory buffers"
iso.3.6.1.2.1.25.2.3.1.3.7 = STRING: "Cached memory"
iso.3.6.1.2.1.25.2.3.1.3.8 = STRING: "Shared memory"
iso.3.6.1.2.1.25.2.3.1.3.10 = STRING: "Swap space"
iso.3.6.1.2.1.25.2.3.1.3.31 = STRING: "/"
iso.3.6.1.2.1.25.2.3.1.3.33 = STRING: "/dev"
iso.3.6.1.2.1.25.2.3.1.3.45 = STRING: "/dev/lxd"
iso.3.6.1.2.1.25.2.3.1.3.46 = STRING: "/dev/.lxd-mounts"
iso.3.6.1.2.1.25.2.3.1.3.62 = STRING: "/proc/sys/kernel/random/boot_id"
iso.3.6.1.2.1.25.2.3.1.3.65 = STRING: "/dev/shm"
iso.3.6.1.2.1.25.2.3.1.3.66 = STRING: "/run"
iso.3.6.1.2.1.25.2.3.1.3.67 = STRING: "/run/lock"
iso.3.6.1.2.1.25.2.3.1.3.68 = STRING: "/sys/fs/cgroup"
iso.3.6.1.2.1.25.2.3.1.3.82 = STRING: "/snap"
iso.3.6.1.2.1.25.2.3.1.3.85 = STRING: "/run/snapd/ns"
iso.3.6.1.2.1.25.2.3.1.3.85 = No more variables left in this MIB View (It is past the end of the MIB tree)

description: updated
Eric Desrochers (slashd) wrote :

I just sponsored 'net-snmp' for Disco. It is now waiting in the upload queue for SRU team approval to start building in disco-proposed for the testing phase.

Eric Desrochers (slashd) wrote :

I just sponsored 'net-snmp' for Bionic. It is now waiting in the upload queue for SRU team approval to start building in bionic-proposed for the testing phase.

The attachment "fix_hrstorage_after_autofs.patch" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
Eric Desrochers (slashd) wrote :

I just sponsored 'net-snmp' for Xenial. It is now waiting in the upload queue for SRU team approval to start building in xenial-proposed for the testing phase.

Eric Desrochers (slashd) on 2019-09-07
description: updated
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package net-snmp - 5.7.3+dfsg-5ubuntu4

---------------
net-snmp (5.7.3+dfsg-5ubuntu4) eoan; urgency=medium

  * d/p/put-paranthesis-around-macros-which-are-expressions.patch:
    - put paranthesis around macros which are expressions.
    (LP: #1843036)

 -- Eric Desrochers <email address hidden> Fri, 06 Sep 2019 23:15:43 +0000

Changed in net-snmp (Ubuntu Eoan):
status: Fix Committed → Fix Released
Eric Desrochers (slashd) on 2019-09-07
description: updated

Hello Eric, or anyone else affected,

Accepted net-snmp into disco-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/net-snmp/5.7.3+dfsg-5ubuntu1.2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-disco to verification-done-disco. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-disco. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in net-snmp (Ubuntu Disco):
status: In Progress → Fix Committed
tags: added: verification-needed verification-needed-disco
Łukasz Zemczak (sil2100) wrote :

Hello Eric, or anyone else affected,

Accepted net-snmp into bionic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/net-snmp/5.7.3+dfsg-1.8ubuntu3.3 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-bionic to verification-done-bionic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-bionic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in net-snmp (Ubuntu Bionic):
status: In Progress → Fix Committed
tags: added: verification-needed-bionic
Łukasz Zemczak (sil2100) wrote :

Hello Eric, or anyone else affected,

Accepted net-snmp into xenial-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/net-snmp/5.7.3+dfsg-1ubuntu4.4 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-xenial to verification-done-xenial. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-xenial. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in net-snmp (Ubuntu Xenial):
status: In Progress → Fix Committed
tags: added: verification-needed-xenial
Eric Desrochers (slashd) wrote :

[VERIFICATION DISCO]

Using proposed package version "5.7.3+dfsg-5ubuntu1.2"

# lsb_release -cs
disco

# snmpwalk -v2c -cpublic localhost 1.3.6.1.2.1.25.2.3.1.3
iso.3.6.1.2.1.25.2.3.1.3.1 = STRING: "Physical memory"
iso.3.6.1.2.1.25.2.3.1.3.3 = STRING: "Virtual memory"
iso.3.6.1.2.1.25.2.3.1.3.6 = STRING: "Memory buffers"
iso.3.6.1.2.1.25.2.3.1.3.7 = STRING: "Cached memory"
iso.3.6.1.2.1.25.2.3.1.3.8 = STRING: "Shared memory"
iso.3.6.1.2.1.25.2.3.1.3.10 = STRING: "Swap space"
iso.3.6.1.2.1.25.2.3.1.3.31 = STRING: "/"
iso.3.6.1.2.1.25.2.3.1.3.33 = STRING: "/dev"
iso.3.6.1.2.1.25.2.3.1.3.45 = STRING: "/dev/lxd"
iso.3.6.1.2.1.25.2.3.1.3.46 = STRING: "/dev/.lxd-mounts"
iso.3.6.1.2.1.25.2.3.1.3.62 = STRING: "/proc/sys/kernel/random/boot_id"
iso.3.6.1.2.1.25.2.3.1.3.65 = STRING: "/dev/shm"
iso.3.6.1.2.1.25.2.3.1.3.66 = STRING: "/run"
iso.3.6.1.2.1.25.2.3.1.3.67 = STRING: "/run/lock"
iso.3.6.1.2.1.25.2.3.1.3.68 = STRING: "/sys/fs/cgroup"
iso.3.6.1.2.1.25.2.3.1.3.82 = STRING: "/snap"
iso.3.6.1.2.1.25.2.3.1.3.85 = STRING: "/run/snapd/ns"
iso.3.6.1.2.1.25.2.3.1.3.85 = No more variables left in this MIB View (It is past the end of the MIB tree)

The above confirms the regression is now fixed !

tags: added: verification-done-disco
removed: verification-needed-disco
Eric Desrochers (slashd) wrote :

[VERIFICATION BIONIC]

Using proposed package version "5.7.3+dfsg-1.8ubuntu3.3"

# lsb_release -cs
bionic

# snmpwalk -v2c -cpublic localhost 1.3.6.1.2.1.25.2.3.1.3
iso.3.6.1.2.1.25.2.3.1.3.1 = STRING: "Physical memory"
iso.3.6.1.2.1.25.2.3.1.3.3 = STRING: "Virtual memory"
iso.3.6.1.2.1.25.2.3.1.3.6 = STRING: "Memory buffers"
iso.3.6.1.2.1.25.2.3.1.3.7 = STRING: "Cached memory"
iso.3.6.1.2.1.25.2.3.1.3.8 = STRING: "Shared memory"
iso.3.6.1.2.1.25.2.3.1.3.10 = STRING: "Swap space"
iso.3.6.1.2.1.25.2.3.1.3.31 = STRING: "/"
iso.3.6.1.2.1.25.2.3.1.3.33 = STRING: "/dev"
iso.3.6.1.2.1.25.2.3.1.3.45 = STRING: "/dev/lxd"
iso.3.6.1.2.1.25.2.3.1.3.46 = STRING: "/dev/.lxd-mounts"
iso.3.6.1.2.1.25.2.3.1.3.63 = STRING: "/proc/sys/kernel/random/boot_id"
iso.3.6.1.2.1.25.2.3.1.3.66 = STRING: "/dev/shm"
iso.3.6.1.2.1.25.2.3.1.3.67 = STRING: "/run"
iso.3.6.1.2.1.25.2.3.1.3.68 = STRING: "/run/lock"
iso.3.6.1.2.1.25.2.3.1.3.69 = STRING: "/sys/fs/cgroup"
iso.3.6.1.2.1.25.2.3.1.3.69 = No more variables left in this MIB View (It is past the end of the MIB tree

tags: added: verification-done-bionic
removed: verification-needed-bionic
Eric Desrochers (slashd) wrote :

[VERIFICATION XENIAL]

Using proposed package version "5.7.3+dfsg-1ubuntu4.4"

# lsb_release -cs
xenial

snmpwalk -v2c -cpublic localhost 1.3.6.1.2.1.25.2.3.1.3
iso.3.6.1.2.1.25.2.3.1.3.1 = STRING: "Physical memory"
iso.3.6.1.2.1.25.2.3.1.3.3 = STRING: "Virtual memory"
iso.3.6.1.2.1.25.2.3.1.3.6 = STRING: "Memory buffers"
iso.3.6.1.2.1.25.2.3.1.3.7 = STRING: "Cached memory"
iso.3.6.1.2.1.25.2.3.1.3.8 = STRING: "Shared memory"
iso.3.6.1.2.1.25.2.3.1.3.10 = STRING: "Swap space"
iso.3.6.1.2.1.25.2.3.1.3.31 = STRING: "/"
iso.3.6.1.2.1.25.2.3.1.3.33 = STRING: "/dev"
iso.3.6.1.2.1.25.2.3.1.3.45 = STRING: "/dev/lxd"
iso.3.6.1.2.1.25.2.3.1.3.46 = STRING: "/dev/.lxd-mounts"
iso.3.6.1.2.1.25.2.3.1.3.63 = STRING: "/proc/sys/kernel/random/boot_id"
iso.3.6.1.2.1.25.2.3.1.3.66 = STRING: "/dev/shm"
iso.3.6.1.2.1.25.2.3.1.3.67 = STRING: "/run"
iso.3.6.1.2.1.25.2.3.1.3.68 = STRING: "/run/lock"
iso.3.6.1.2.1.25.2.3.1.3.69 = STRING: "/sys/fs/cgroup"
iso.3.6.1.2.1.25.2.3.1.3.69 = No more variables left in this MIB View (It is past the end of the MIB tree)

tags: added: verification-done-xenial verification-needed-bionic
removed: verification-done-bionic verification-needed-xenial
Eric Desrochers (slashd) wrote :

Hi,

I see there is at least 26 ppl impacted by this regression according to the LP bug.

Please feel free to test the -proposed package and provide positive feedback. If we have enough feedback by Thursday, I'll do my best to have an early release.

Regards,
Eric

Eric Lacroix (nealacroix) wrote :

Hello,

I have installed the proposed update here : https://launchpad.net/ubuntu/+source/net-snmp/5.7.3+dfsg-1.8ubuntu3.3/+build/17742932
on one of Bionic servers and the Nagios monitoring has not raised any errors or warnings.

Sorry I can't help you more than this.

Best,
Eric

Benjamin Weik (bweik) wrote :

Hi,

installed

libsnmp-base/bionic-proposed,bionic-proposed 5.7.3+dfsg-1.8ubuntu3.3 all [upgradable from: 5.7.3+dfsg-1.8ubuntu3.2]
libsnmp30/bionic-proposed 5.7.3+dfsg-1.8ubuntu3.3 amd64 [upgradable from: 5.7.3+dfsg-1.8ubuntu3.2]
snmpd/bionic-proposed 5.7.3+dfsg-1.8ubuntu3.3 amd64 [upgradable from: 5.7.3+dfsg-1.8ubuntu3.2]

from proposed repo and an confirm that disks haven been restored.

David Ford (atheros1200) wrote :

Confirmed fixed in Xenial

ii libsnmp-base 5.7.3+dfsg-1ubuntu4.4 all SNMP configuration script, MIBs and documentation
ii libsnmp30:amd64 5.7.3+dfsg-1ubuntu4.4 amd64 SNMP (Simple Network Management Protocol) library
ii snmp 5.7.3+dfsg-1ubuntu4.4 amd64 SNMP (Simple Network Management Protocol) applications
ii snmpd 5.7.3+dfsg-1ubuntu4.4 amd64 SNMP (Simple Network Management Protocol) agents

HOST-RESOURCES-MIB::hrStorageDescr.1 = STRING: Physical memory
HOST-RESOURCES-MIB::hrStorageDescr.3 = STRING: Virtual memory
HOST-RESOURCES-MIB::hrStorageDescr.6 = STRING: Memory buffers
HOST-RESOURCES-MIB::hrStorageDescr.7 = STRING: Cached memory
HOST-RESOURCES-MIB::hrStorageDescr.8 = STRING: Shared memory
HOST-RESOURCES-MIB::hrStorageDescr.10 = STRING: Swap space
HOST-RESOURCES-MIB::hrStorageDescr.31 = STRING: /
HOST-RESOURCES-MIB::hrStorageDescr.37 = STRING: /run
HOST-RESOURCES-MIB::hrStorageDescr.39 = STRING: /dev/shm
HOST-RESOURCES-MIB::hrStorageDescr.40 = STRING: /run/lock
HOST-RESOURCES-MIB::hrStorageDescr.41 = STRING: /sys/fs/cgroup
HOST-RESOURCES-MIB::hrStorageDescr.61 = STRING: /run/user/2143

David Ford (atheros1200) wrote :

libsnmp30:
  Installed: 5.7.3+dfsg-1ubuntu4.4
  Candidate: 5.7.3+dfsg-1ubuntu4.4
  Version table:
 *** 5.7.3+dfsg-1ubuntu4.4 500
        500 http://archive.ubuntu.com/ubuntu xenial-proposed/main amd64 Packages

Chris Taylor (ctaylor-g) wrote :

Worked for me on Xenial. Nagios also reporting correctly.

ii libsnmp-base 5.7.3+dfsg-1ubuntu4.4 all SNMP configuration script, MIBs and documentation
ii libsnmp30:amd64 5.7.3+dfsg-1ubuntu4.4 amd64 SNMP (Simple Network Management Protocol) library
ii snmpd 5.7.3+dfsg-1ubuntu4.4 amd64 SNMP (Simple Network Management Protocol) agents

Thank you!

Joseph Gooch (whitemystic) wrote :

Confirmed working Xenial:
snmpd:
  Installed: 5.7.3+dfsg-1ubuntu4.4
  Candidate: 5.7.3+dfsg-1ubuntu4.4
  Version table:
 *** 5.7.3+dfsg-1ubuntu4.4 500
        500 http://us.archive.ubuntu.com/ubuntu xenial-proposed/main amd64 Packages
        100 /var/lib/dpkg/status

And Bionic:
snmpd:
  Installed: 5.7.3+dfsg-1.8ubuntu3.3
  Candidate: 5.7.3+dfsg-1.8ubuntu3.3
  Version table:
 *** 5.7.3+dfsg-1.8ubuntu3.3 500
        500 http://us.archive.ubuntu.com/ubuntu bionic-proposed/main amd64 Packages
        100 /var/lib/dpkg/status

Chris Newcomer (cnewcomer) wrote :

VALIDATION: XENIAL: PASSED

See attached typescript output.

The snmpd package works as intended on Xenial. The snmpwalk returns the data expected and it doesn't force direct-mapped autofs filesystems to mount every time you run the snmpwalk.

Chris Newcomer (cnewcomer) wrote :

VALIDATION: BIONIC: PASSED

See attached typescript output.

The snmpd package works as intended on Bionic. The snmpwalk returns the data expected and it doesn't force direct-mapped autofs filesystems to mount every time you run the snmpwalk.

Chris Newcomer (cnewcomer) wrote :

VALIDATION: DISCO: PASSED

See attached typescript output.

The snmpd package works as intended on Disco. The snmpwalk returns the data expected and it doesn't force direct-mapped autofs filesystems to mount every time you run the snmpwalk.

Eric Desrochers (slashd) wrote :

Thanks a bunch all ! Your collaboration is much appreciated !

I'll sync with the SRU verification team later this week and release it by EOW, if no negative feedback is raise by then.

Regards,
Eric

tags: added: verification-done-bionic
removed: verification-needed-bionic
Eric Desrochers (slashd) on 2019-09-10
description: updated
Changed in net-snmp (Ubuntu Bionic):
status: Fix Committed → Fix Released
Eric Desrochers (slashd) on 2019-09-11
Changed in net-snmp (Ubuntu Bionic):
status: Fix Released → Fix Committed
Łukasz Zemczak (sil2100) wrote :

Thanks for all the verification! I will quickly sync-up with Eric today and, if all looks good, I'll release this update early, without the 7-day aging period, today.

Eric Desrochers (slashd) wrote :

@Łukasz Zemczak (sil2100),

All good on my side.

- Eric

The verification of the Stable Release Update for net-snmp has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Launchpad Janitor (janitor) wrote :

This bug was fixed in the package net-snmp - 5.7.3+dfsg-5ubuntu1.2

---------------
net-snmp (5.7.3+dfsg-5ubuntu1.2) disco; urgency=medium

  * d/p/put-paranthesis-around-macros-which-are-expressions.patch:
    - put paranthesis around macros which are expressions.
    (LP: #1843036)

  * d/p/fix-check-hr-filesys-autofs.patch:
    - On Linux getmntent() is available but getfsstat() not.
      Hence remove #if HAVE_GETFSSTAT from around the HRFS_type
      check.

net-snmp (5.7.3+dfsg-5ubuntu1.1) disco; urgency=medium

  * Skip autofs entries when calling statfs to prevent autofs being mounted on
    snmp startup (LP: #1835818):
    - d/p/autofs-skip-autofs-entries.patch
    - d/p/autofs-fix-a-recently-introduced-bug.patch

 -- Eric Desrochers <email address hidden> Fri, 06 Sep 2019 23:44:49 +0000

Changed in net-snmp (Ubuntu Disco):
status: Fix Committed → Fix Released
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package net-snmp - 5.7.3+dfsg-1.8ubuntu3.3

---------------
net-snmp (5.7.3+dfsg-1.8ubuntu3.3) bionic; urgency=medium

  * d/p/put-paranthesis-around-macros-which-are-expressions.patch:
    - put paranthesis around macros which are expressions.
    (LP: #1843036)

  * d/p/fix-check-hr-filesys-autofs.patch:
    - On Linux getmntent() is available but getfsstat() not.
      Hence remove #if HAVE_GETFSSTAT from around the HRFS_type
      check.

net-snmp (5.7.3+dfsg-1.8ubuntu3.2) bionic; urgency=medium

  * Skip autofs entries when calling statfs to prevent autofs being mounted on
    snmpd startup (LP: #1835818):
    - d/p/autofs-skip-autofs-entries.patch
    - d/p/autofs-fix-a-recently-introduced-bug.patch

 -- Eric Desrochers <email address hidden> Fri, 06 Sep 2019 23:59:19 +0000

Changed in net-snmp (Ubuntu Bionic):
status: Fix Committed → Fix Released
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package net-snmp - 5.7.3+dfsg-1ubuntu4.4

---------------
net-snmp (5.7.3+dfsg-1ubuntu4.4) xenial; urgency=medium

  * d/p/put-paranthesis-around-macros-which-are-expressions.patch:
    - put paranthesis around macros which are expressions.
    (LP: #1843036)

  * d/p/fix-check-hr-filesys-autofs.patch:
    - On Linux getmntent() is available but getfsstat() not.
      Hence remove #if HAVE_GETFSSTAT from around the HRFS_type
      check.

net-snmp (5.7.3+dfsg-1ubuntu4.3) xenial; urgency=medium

  * Skip autofs entries when calling statfs to prevent autofs being mounted on
    snmpd startup (LP: #1835818):
    - d/p/autofs-skip-autofs-entries.patch
    - d/p/autofs-fix-a-recently-introduced-bug.patch

 -- Eric Desrochers <email address hidden> Sat, 07 Sep 2019 00:07:32 +0000

Changed in net-snmp (Ubuntu Xenial):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers