Image cache has virtually no unit test coverage

Bug #872372 reported by Jay Pipes
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
Fix Released
Medium
Jay Pipes

Bug Description

The image cache does not have an unit tests, and there is quite a bit of the code that can be easily unit tested.

Tags: cache testing
Revision history for this message
Jay Pipes (jaypipes) wrote :
Jay Pipes (jaypipes)
Changed in glance:
status: Confirmed → In Progress
assignee: nobody → Jay Pipes (jaypipes)
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix merged to glance (master)

Reviewed: https://review.openstack.org/923
Committed: http://github.com/openstack/glance/commit/ad9e9ca3f741af714b9faf46224bf12e4a2693bd
Submitter: Jenkins
Branch: master

 status fixcommitted
 done

commit ad9e9ca3f741af714b9faf46224bf12e4a2693bd
Author: Jay Pipes <email address hidden>
Date: Tue Oct 18 09:27:27 2011 -0400

    Overhauls the image cache to be truly optional

    Fixes LP Bug#874580 - keyerror 'location' when fetch errors
    Fixes LP Bug#817570 - Make new image cache a true extension
    Fixes LP Bug#872372 - Image cache has virtually no unit test coverage

    * Adds unit tests for the image cache (coverage goes from 26% to 100%)
    * Removes caching logic from the images controller and places it into
      a removeable transparent caching middleware
    * Adds a functional test case that verifies caching of an image
      and subsequent cache hits
    * Removes the image_cache_enabled configuration variable, since it's
      now enabled by simply including the cache in the application
      pipeline
    * Adds a singular glance-cache.conf to etc/ that replaces the
      multiple glance-pruner.conf, glance-reaper.conf and
      glance-prefetcher.conf files
    * Adds documentation on enabling and configuring the image cache

    TODO: Add documentation on the image cache utilities, like reaper,
          prefetcher, etc.

    Change-Id: I58845871deee26f81ffabe1750adc472ce5b3797

Changed in glance:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in glance:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in glance:
milestone: essex-1 → 2012.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.