missing dependency on cinderclient

Bug #1074085 reported by Ionuț Arțăriși
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Sean Dague

Bug Description

Running the testsuite without installing the test-requires gave me the following error:

File "/home/mapleoin/openstack/nova/nova/volume/cinder.py", line 24, in <module>
    from cinderclient import service_catalog
ImportError: No module named cinderclient

It appears there's a cinderclient import in nova/volume/cinder.py, but there is no python-cinderclient dependency in tools/pip-requires. The dependency is only present in tools/test-requires (which is why this bug went uncaught by the testsuites so far I guess).

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.openstack.org/15267

Changed in nova:
assignee: nobody → Ionuț Arțăriși (mapleoin)
status: New → In Progress
Changed in nova:
assignee: Ionuț Arțăriși (mapleoin) → Sean Dague (sdague-b)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/folsom)

Fix proposed to branch: stable/folsom
Review: https://review.openstack.org/15278

tags: added: folsom-backport-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: stable/folsom
Review: https://review.openstack.org/15279

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/15132
Committed: http://github.com/openstack/nova/commit/9e03eab1884107d04e0b630dfe5a57f166066232
Submitter: Jenkins
Branch: master

commit 9e03eab1884107d04e0b630dfe5a57f166066232
Author: Sean Dague <email address hidden>
Date: Wed Oct 31 14:19:49 2012 -0400

    move python-cinderclient to pip-requires

    python-cinderclient is now actually a requirement, move
    this from test-requires to pip-requires to reflect that.

    Fixes bug #1074085

    Change-Id: Iea6c58faab2462d137049d690d58d951bd4e2c9e

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → grizzly-1
status: Fix Committed → Fix Released
tags: removed: folsom-backport-potential
Changed in nova:
importance: Undecided → Low
Thierry Carrez (ttx)
Changed in nova:
milestone: grizzly-1 → 2013.1
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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