Bug 1520637 - python-rtslib is missing RPM dependency on python-pyudev in ceph-tools repos
Summary: python-rtslib is missing RPM dependency on python-pyudev in ceph-tools repos
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Ceph Storage
Classification: Red Hat Storage
Component: iSCSI
Version: 2.4
Hardware: Unspecified
OS: Linux
low
low
Target Milestone: z3
: 3.0
Assignee: Jason Dillaman
QA Contact: Madhavi Kasturi
Aron Gunn
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-12-04 21:17 UTC by Alan Bishop
Modified: 2018-05-15 18:21 UTC (History)
11 users (show)

Fixed In Version: python-rtslib-2.1.fb64-3.el7cp
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1527605 (view as bug list)
Environment:
Last Closed: 2018-05-15 18:20:29 UTC
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Bugzilla 1527605 0 high CLOSED python-rtslib is missing RPM dependency on python-pyudev in ceph-tools repos 2021-02-22 00:41:40 UTC
Red Hat Product Errata RHBA-2018:1563 0 None None None 2018-05-15 18:21:15 UTC

Internal Links: 1527605

Description Alan Bishop 2017-12-04 21:17:04 UTC
The python-rtslib package requires python-pyudev, and that dependency is
properly noted in the RPM in the rhel-7.4 repo. However, the corresponding
RPMs in the ceph-2-rhel-7 and ceph-3.0-rhel-7 ceph-tools repos are missing
that dependency.

This issue is affecting containers built for Openstack. Specifically, when the
cinder-volume container is built, it pulls in python-rtslib from the ceph-2
ceph-tools repo. But when the container runs, it crashes because the
python-pyudev package is missing.

Comment 3 Ken Dreyer (Red Hat) 2017-12-04 21:26:04 UTC
Alan, would you please provide "Steps to Reproduce", "Actual Results", "Expected Results" ?

Comment 4 Ken Dreyer (Red Hat) 2017-12-04 21:27:14 UTC
Targeting this to RHCEPH 3 so we don't regress. We'll need an RHCEPH 2 clone if we need to fix this there also (discussion ongoing with PM)

Comment 5 Alan Bishop 2017-12-04 22:06:55 UTC
Steps to reproduce:

1) start with a clean rhel-7.4 install
2) make sure python-pyudev is removed

$ yum remove python-pyudev

3) enable rhcs-2 ceph-tools repo
4) install python-rtslib

$ yum install python-rtslib

(not it will not pull in python-pyudev as a dependency)

5) test using python interpreter

$ python
Python 2.7.5 (default, May  3 2017, 07:55:04) 
[GCC 4.8.5 20150623 (Red Hat 4.8.5-14)] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import rtslib

Actual results:

>>> import rtslib
__main__:1: UserWarning: 'rtslib' package name for rtslib-fb is deprecated, please instead import 'rtslib_fb'
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python2.7/site-packages/rtslib/__init__.py", line 24, in <module>
    from .root import RTSRoot
  File "/usr/lib/python2.7/site-packages/rtslib/root.py", line 25, in <module>
    from .node import CFSNode
  File "/usr/lib/python2.7/site-packages/rtslib/node.py", line 23, in <module>
    from .utils import fread, fwrite, RTSLibError, RTSLibNotInCFS
  File "/usr/lib/python2.7/site-packages/rtslib/utils.py", line 30, in <module>
    import pyudev
ImportError: No module named pyudev

Expected results:

No error when importing rtslib.


Note: try the same test w/o the rhcs-2 ceph-tools repo enabled, and when you install python-rtslib it should automatically install python-pyudev.

Comment 6 Ken Dreyer (Red Hat) 2017-12-06 17:57:52 UTC
Alan, I heard this week that OSP 12's cinder-volume-docker image pulls in python-rtslib from the RHCEPH Tools repo, and this leads to this bug. Is that right? Would you please say more about the priority for OSP 12?

Comment 7 Ken Dreyer (Red Hat) 2017-12-06 17:58:49 UTC
Oh, you already mentioned this in the description above, sorry :)

What is the impact to OSP 12's Cinder? Is there a workaround? Trying to get the Priority field set...

Comment 8 Alan Bishop 2017-12-07 21:20:44 UTC
Ken,

This will not hold up OSP-12 GA, but a fix will be required ASAP, hopefully in line with the first OSP-12z. CC'ing a few others who may have a firmer view.

Comment 9 Ken Dreyer (Red Hat) 2017-12-07 21:55:12 UTC
What is the current workaround for cinder-volume-docker?

Comment 10 Paul Grist 2018-01-10 02:33:35 UTC
Hi Ken,

I wanted to get an update here. First this issue was indeed critical and would block 3rd party cinder container efforts which need to be developed by our partners on OSP12 in order to be ready for OSP13 GA.  The current work around required partner to layer the fix onto the cinder container.

The cinder team pursued an alternative work-around/fix for 12z1 in https://bugzilla.redhat.com/show_bug.cgi?id=1527605

So this one still should get fixes, but is no longer urgent.

thanks,
Paul

Comment 17 errata-xmlrpc 2018-05-15 18:20:29 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2018:1563


Note You need to log in before you can comment on or make changes to this bug.