get_info environ caching is leaky and brittle

Bug #1501939 reported by clayg
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Object Storage (swift)
Fix Released
Medium
Unassigned

Bug Description

Everytime we look at the get_info caching it seems like we should clean it up. We've even had the occasional bug slip through because it's hard to reason about what's going on in that sub-system because it crosses so many abstraction boundaries. Sometimes it just breaks down in surprising ways because when making unrelated changes.

Also it doesn't share cached data between subrequests.

A couple of good fixes popped up when trying to work through an unrelated change in lp bug #1449212 but they never got merged.

Some highlights:

 * shallow-copy the WSGI environment
 * do not rely on the proxy server setting the cache in the get_*_info() request environ, and instead use the response directly

... maybe some other good ideas, hopefully others can backfill all the stuff we wanted to fix in an itemized list for when we get around to trying to fix this on purpose.

Changed in swift:
importance: Undecided → Medium
Changed in swift:
status: New → Confirmed
Revision history for this message
Mahati Chamarthy (mahati-chamarthy) wrote :

Is there a way to reproduce the problems with get_info caching?

Revision history for this message
Thiago da Silva (thiagodasilva) wrote :

I think this bug was fixed with the following patches:
https://review.openstack.org/#/c/280977/
https://review.openstack.org/#/c/280978/

Revision history for this message
clayg (clay-gerrard) wrote :

YES this was fixed!

Changed in swift:
status: Confirmed → Fix Released
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.