Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 888082

Summary: Glance should depend on python-qpid
Product: Red Hat OpenStack Reporter: Russell Bryant <rbryant>
Component: openstack-glanceAssignee: RHOS Maint <rhos-maint>
Status: CLOSED WONTFIX QA Contact: Yaniv Kaul <ykaul>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 2.1CC: apevec, eglynn, jkt
Target Milestone: ---Keywords: Triaged
Target Release: 2.1   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-12-21 18:14:23 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 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.