Latest keystoneclient breaks tests

Bug #1347319 reported by Steven Hardy
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
OpenStack Heat
Fix Released
High
Steven Hardy

Bug Description

traceback-1: {{{
Traceback (most recent call last):
  File "/usr/lib/python2.7/site-packages/mox.py", line 284, in VerifyAll
    mock_obj._Verify()
  File "/usr/lib/python2.7/site-packages/mox.py", line 504, in _Verify
    raise ExpectedMethodCallsError(self._expected_calls_queue)
ExpectedMethodCallsError: Verify: Expected methods never called:
  0. DomainManager.get('adomain123') -> None
}}}

Traceback (most recent call last):
  File "heat/tests/test_heatclient.py", line 1430, in test_enable_stack_domain_user
    self._stub_domain_admin_client_domain_get()
  File "heat/tests/test_heatclient.py", line 1375, in _stub_domain_admin_client_domain_get
    loaded=True))
  File "/opt/stack/python-keystoneclient/keystoneclient/openstack/common/apiclient/base.py", line 440, in __init__
    self._init_completion_cache()
  File "/opt/stack/python-keystoneclient/keystoneclient/openstack/common/apiclient/base.py", line 443, in _init_completion_cache
    cache_write = getattr(self.manager, 'write_to_completion_cache', None)
  File "/usr/lib/python2.7/site-packages/mox.py", line 606, in __getattr__
    raise UnknownMethodCallError(name)
UnknownMethodCallError: Method called is not a member of the object: write_to_completion_cache

Steven Hardy (shardy)
Changed in heat:
importance: Undecided → High
status: New → In Progress
assignee: nobody → Steven Hardy (shardy)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to heat (master)

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

Steven Hardy (shardy)
Changed in heat:
milestone: none → juno-2
Revision history for this message
Thomas Herve (therve) wrote :

I approved the patch so that we can unblock the gate, but I think we could do:

self.mock_ks_v3_client_domain_mngr.write_to_completion_cache = None

And keep using the Domain class. What do you think?

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

Reviewed: https://review.openstack.org/108875
Committed: https://git.openstack.org/cgit/openstack/heat/commit/?id=e81ba6f2463dedb7dff753bc256f817be61ab184
Submitter: Jenkins
Branch: master

commit e81ba6f2463dedb7dff753bc256f817be61ab184
Author: Steven Hardy <email address hidden>
Date: Wed Jul 23 02:43:43 2014 +0100

    test_heatclient fix mocking with latest keystone

    The tests break with latest (0.10.0) keystoneclient, so replace
    the mocked domain with a MockAnything object

    Change-Id: Icd460e554656e11ef8c4bcda62c1363b7eed8a68
    Closes-Bug: #1347319

Changed in heat:
status: In Progress → Fix Committed
Changed in heat:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in heat:
milestone: juno-2 → 2014.2
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.