Bug 1520637

Summary:	python-rtslib is missing RPM dependency on python-pyudev in ceph-tools repos
Product:	[Red Hat Storage] Red Hat Ceph Storage	Reporter:	Alan Bishop <abishop>
Component:	iSCSI	Assignee:	Jason Dillaman <jdillama>
Status:	CLOSED ERRATA	QA Contact:	Madhavi Kasturi <mkasturi>
Severity:	low	Docs Contact:	Aron Gunn <agunn>
Priority:	low
Version:	2.4	CC:	abishop, agunn, ceph-eng-bugs, ceph-qe-bugs, cschwede, jdillama, kdreyer, pgrist, scohen, tchandra, tvignaud
Target Milestone:	z3
Target Release:	3.0
Hardware:	Unspecified
OS:	Linux
Whiteboard:
Fixed In Version:	python-rtslib-2.1.fb64-3.el7cp	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:
Clones:	1527605 (view as bug list)		Environment:
Last Closed:	2018-05-15 18:20:29 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

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