Test suite fails with latest novaclient

Bug #958857 reported by Gabriel Hurley on 2012-03-18
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Critical
Gabriel Hurley

Bug Description

Running the test suite with the latest novaclient results in this traceback:

Traceback (most recent call last):
  File "/srv/horizon/horizon/horizon/test.py", line 77, in setUp
    load_test_data(self)
  File "/srv/horizon/horizon/horizon/tests/test_data/utils.py", line 29, in load_test_data
    data_func(load_onto)
  File "/srv/horizon/horizon/horizon/tests/test_data/nova_data.py", line 161, in data
    'OS-FLV-EXT-DATA:ephemeral': 0})
  File "/srv/horizon/horizon/.venv/src/python-novaclient/novaclient/base.py", line 305, in __init__
    self.manager.write_to_completion_cache('human_id', human_id)
TypeError: unbound method write_to_completion_cache() must be called with FlavorManager instance as first argument (got str instance instead)

Reviewed: https://review.openstack.org/5488
Committed: http://github.com/openstack/horizon/commit/b53d08386fa3d0b054844f8e9b6a1a05b744c6af
Submitter: Jenkins
Branch: master

commit b53d08386fa3d0b054844f8e9b6a1a05b744c6af
Author: Gabriel Hurley <email address hidden>
Date: Sun Mar 18 15:30:45 2012 -0700

    Instantiate novaclient managers for test data.

    Some novaclient resources now have a "human_id" property
    which requires a call to a bound method on the manager. Thereby
    we need to instantiate the managers in the test data rather than
    simply passing the class.

    Fixes bug 958857.

    There's also some cleanup on the test settings here, to remove
    some outdated settings and to use an in-memory database instead
    of a file on disk.

    Change-Id: I1331e4ae5658b45d01b8247623554c5c43f67578

Changed in horizon:
status: In Progress → Fix Committed
Thierry Carrez (ttx) on 2012-03-20
Changed in horizon:
status: Fix Committed → Fix Released
Thierry Carrez (ttx) on 2012-04-05
Changed in horizon:
milestone: essex-rc1 → 2012.1
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers