[SRU] python-pysnmp4 pkg inconsistent in cosmic

Bug #1806474 reported by Corey Bryant on 2018-12-03
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ceilometer (Ubuntu)
Undecided
Unassigned
Cosmic
High
Unassigned
python-pysnmp4 (Ubuntu)
Undecided
Unassigned
Cosmic
High
Unassigned

Bug Description

[Impact]
I hit the following error when running ceilometer unit tests for cosmic:

=========================
Failures during discovery
=========================
--- import errors ---
Failed to import test module: ceilometer.tests.unit.hardware.inspector.test_snmp
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/unittest2/loader.py", line 456, in _find_test_path
    module = self._get_module_from_name(name)
  File "/usr/lib/python2.7/dist-packages/unittest2/loader.py", line 395, in _get_module_from_name
    __import__(name)
  File "ceilometer/tests/unit/hardware/inspector/test_snmp.py", line 23, in <module>
    from ceilometer.hardware.inspector import snmp
  File "ceilometer/hardware/inspector/snmp.py", line 21, in <module>
    from pysnmp.entity.rfc3413.oneliner import cmdgen
ImportError: No module named oneliner

It turns out the tarball that is pulled from git to build pysnmp4 is missing files, such as pysnmp.entity.rfc3413.oneliner. The release artifact from pypi.debian.net is not missing the oneliner directory.

[Test Case]
Building the ceilometer package for rocky should be successful with a good version of pysnmp. I also plan to run tempest regression tests on a full openstack deployment to help ensure there are no regressions.

[Regression Potential]
The current state of pysnmp4 at 4.4.6-1 is incomplete, therefore the diff from 4.4.6-1 to 4.4.6+repack1-1 is not minimal. Most of the diff, however, is added files/directories that were missing in 4.4.6-1. Note that 4.4.6+repack1-1 is already in Debian unstable and the fix for this bug is to sync from unstable. This version has been in unstable for 2 months which should have provided some regression coverage.

Corey Bryant (corey.bryant) wrote :

This bug was already fixed in Debian, however this was fixed after cosmic was released via a +repack in sid:

debian:
 python-pysnmp4 | 4.4.6+repack1-1 | unstable | source, all
ubuntu:
 python-pysnmp4 | 4.4.6-1 | cosmic | source, all
 python-pysnmp4 | 4.4.6+repack1-1 | disco | source
 python-pysnmp4 | 4.4.6+repack1-1 | disco/universe | all

This was fixed via the following bug:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=910269

Can we sync from unstable to cosmic at this point to fix this?

Changed in python-pysnmp4 (Ubuntu Cosmic):
status: New → Triaged
importance: Undecided → High
Changed in python-pysnmp4 (Ubuntu):
status: New → Invalid
summary: - missing oneliner
+ [SRU] python-pysnmp4 pkg inconsistent in cosmic
Changed in ceilometer (Ubuntu):
status: New → Invalid
Changed in ceilometer (Ubuntu Cosmic):
status: New → Triaged
importance: Undecided → High
description: updated

Hello Corey, or anyone else affected,

Accepted python-pysnmp4 into cosmic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/python-pysnmp4/4.4.6+repack1-0ubuntu0.18.10.1 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-cosmic to verification-done-cosmic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-cosmic. 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 python-pysnmp4 (Ubuntu Cosmic):
status: Triaged → Fix Committed
tags: added: verification-needed verification-needed-cosmic
Brian Murray (brian-murray) wrote :

Hello Corey, or anyone else affected,

Accepted ceilometer into cosmic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/ceilometer/1:11.0.1-0ubuntu1 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-cosmic to verification-done-cosmic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-cosmic. 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 ceilometer (Ubuntu Cosmic):
status: Triaged → Fix Committed
Corey Bryant (corey.bryant) wrote :

Testing was successful.

Ceilometer now builds successfully: https://launchpad.net/ubuntu/+source/ceilometer/1:11.0.1-0ubuntu1

And OpenStack regression tests passed for cosmic-rocky-proposed:

======
Totals
======
Ran: 92 tests in 1052.7056 sec.
 - Passed: 84
 - Skipped: 8
 - Expected Fail: 0
 - Unexpected Success: 0
 - Failed: 0
Sum of execute time for each test: 548.8815 sec.

tags: added: verification-done verification-done-cosmic
removed: verification-needed verification-needed-cosmic
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.