python-glanceclient block forever with http

Bug #1822052 reported by Feng Wen
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance Client
Fix Released
Cyril Roelandt

Bug Description

In python-glanceclient, the timeout parameter doesn't work (for example: cinder.glance_request_timeout).

This causes cinder/nova to be blocked forever in case of bad network.

In glanceclient/common/, the timeout parameter is only assigned, but it is not passed to the session at the time of the request.

Did I miss something?

Revision history for this message
Feng Wen (cnwenf) wrote :

python-glanceclient version: stein

Changed in python-glanceclient:
assignee: nobody → Cyril Roelandt (cyril-roelandt)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-glanceclient (master)

Fix proposed to branch: master

Changed in python-glanceclient:
status: New → In Progress
Revision history for this message
Cyril Roelandt (cyril-roelandt) wrote :

This makes sense,I proposed the above fix so that we can discuss it over a review :)

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

Submitter: Zuul
Branch: master

commit d589aa257d3ab99a50444ff42f43b3b9d650d089
Author: Cyril Roelandt <email address hidden>
Date: Tue Apr 9 18:42:52 2019 +0200

    HTTPClient: actually set a timeout for requests

    The 'timeout' attribute was previously left unused.

    Change-Id: If615c390302425fe5a646b4651ec6f56aa08fd22
    Closes-Bug: #1822052

Changed in python-glanceclient:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/python-glanceclient 2.17.0

This issue was fixed in the openstack/python-glanceclient 2.17.0 release.

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.