We should cache find request to avoid multiple requests on find_resource

Bug #1429103 reported by Gorka Eguileor
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-cinderclient
Invalid
Wishlist
Gorka Eguileor

Bug Description

Currently we are generating multiple requests to volumes/detail?all_tenants=1 from find_resource when we could limit this to 1 request and then search the results multiple times.

We can see it if we do:
   cinder --debug show fake_name

We'll see multiple requests that retrieve the exact same data.

Gorka Eguileor (gorka)
Changed in python-cinderclient:
assignee: nobody → Gorka Eguileor (gorka)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-cinderclient (master)

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

Mike Perez (thingee)
Changed in python-cinderclient:
milestone: none → 1.2.0
importance: Undecided → Wishlist
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on python-cinderclient (master)

Change abandoned by Gorka Eguileor (<email address hidden>) on branch: master
Review: https://review.openstack.org/162139

Changed in python-cinderclient:
milestone: 1.2.0 → none
Revision history for this message
Sean McGinnis (sean-mcginnis) wrote :

Gorka, is this still applicable.

Revision history for this message
Sean McGinnis (sean-mcginnis) wrote :

I'm not able to reproduce this with the current patch. Closing as it looks like this may have been fixed by another patch. If not and more work is needed, please feel free to reopen with any additional details.

Changed in python-cinderclient:
status: In Progress → Invalid
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.