Cinder client is not mocked in tests

Bug #1259533 reported by Cristian A Sanchez
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Fix Released
Undecided
Cristian A Sanchez

Bug Description

When running the Horizon test cases, I get the following lines in the console:

Trying your patch leads me to: INFO:urllib3.connectionpool:Starting new HTTP connection (1): public.nova.example.com DEBUG:cinderclient.client:Connection refused: HTTPConnectionPool(host='public.nova.example.com', port=8776): Max retries exceeded with url: /v1/limits (Caused by <class 'socket.gaierror'>: [Errno -2] Name or service not known) .INFO:urllib3.connectionpool:Starting new HTTP connection (1): public.nova.example.com DEBUG:cinderclient.client:Connection refused: HTTPConnectionPool(host='public.nova.example.com', port=8776): Max retries exceeded with url: /v1/limits (Caused by <class 'socket.gaierror'>: [Errno -2] Name or service not known) .INFO:urllib3.connectionpool:Starting new HTTP connection (1): public.nova.example.com DEBUG:cinderclient.client:Connection refused: HTTPConnectionPool(host='public.nova.example.com', port=8776): Max retries exceeded with url: /v1/limits (Caused by <class 'socket.gaierror'>: [Errno -2] Name or service not known) .INFO:urllib3.connectionpool:Starting new HTTP connection (1): public.nova.example.com DEBUG:cinderclient.client:Connection refused: HTTPConnectionPool(host='public.nova.example.com', port=8776): Max retries exceeded with url: /v1/limits (Caused by <class 'socket.gaierror'>: [Errno -2] Name or service not known)

It looks like Cinder client is not mocked.

Changed in horizon:
assignee: nobody → Cristian A Sanchez (cristian-a-sanchez)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

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

Changed in horizon:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to horizon (master)

Reviewed: https://review.openstack.org/61208
Committed: https://git.openstack.org/cgit/openstack/horizon/commit/?id=c86918f90cd257dd0af2e03f5511d21d57702f88
Submitter: Jenkins
Branch: master

commit c86918f90cd257dd0af2e03f5511d21d57702f88
Author: Cristian A Sanchez <email address hidden>
Date: Mon Dec 9 20:34:31 2013 -0800

    Adds Cinder client mock to tests

    Test cases now have a mock for the method tenant_absolute_limits
    in cinder client. The absence of this mock was causing real HTTP
    requests from this client.

    Change-Id: I8b9c4f5affc43e2649f3ebf08580bb3d3687e09b
    Closes-Bug: #1259533

Changed in horizon:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in horizon:
milestone: none → icehouse-2
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in horizon:
milestone: icehouse-2 → 2014.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.