Bug 888082 - Glance should depend on python-qpid
Summary: Glance should depend on python-qpid
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat OpenStack
Classification: Red Hat
Component: openstack-glance
Version: 2.1
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
: 2.1
Assignee: RHOS Maint
QA Contact: Yaniv Kaul
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-12-18 00:12 UTC by Russell Bryant
Modified: 2013-07-04 02:55 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-12-21 18:14:23 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Russell Bryant 2012-12-18 00:12:34 UTC
Description of problem:

When installing glance, python-qpid is not pulled in as a dependency.  python-qpid is required to use the qpid_notifier for message based notifications so it seems like it should be a dependency.


Version-Release number of selected component (if applicable):

openstack-glance-2012.2.1-1.el6ost.noarch
python-glance-2012.2.1-1.el6ost.noarch

Comment 2 Alan Pevec 2012-12-21 17:43:44 UTC
(In reply to comment #0)
> When installing glance, python-qpid is not pulled in as a dependency. 
> python-qpid is required to use the qpid_notifier for message based
> notifications so it seems like it should be a dependency.

Distributed glance-api.conf has
notifier_strategy = noop
so it's not a dependency out-of-the-box (that's the thinking)

Would you recommend changing this default to "qpid" and then add python-qpid dependency?

Comment 3 Eoghan Glynn 2012-12-21 18:14:23 UTC
Yes I'd agree with Alan there. I'd see glance's AMQP usage as a special case, given that it's essentially optional as glance does not depend on RPC for intra-process communication in the same way as say nova or cinder.

Once we've packaged up something else that has a stronger dependency on glance notifications being enabled (I'm thinking ceilo here), then we could consider adding the dependency on qpid for glance and setting the notifier_strategy out-of-the-box.

Marking as WONTFIX for now, though please feel free to reopen.


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