lp:~jaypipes/glance/bug818292

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:~hudson-openstack/glance/trunk at revision 180

Rick Harris (community): Approve on 2011-08-03

Matt Dietz (community): Approve on 2011-08-03

Josh Kearney (community): Approve on 2011-08-03

Diff: 99 lines (+39/-7)

4 files modified

glance/api/v1/images.py (+0/-1)
glance/store/s3.py (+35/-4)
glance/tests/functional/test_s3.py (+2/-1)
glance/tests/unit/test_s3_store.py (+2/-1)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #818292: Unable to register image with FS backend

Critical

Fix Released

Link a bug report

Related blueprints

181. By Jay Pipes on 2011-08-03

Make sure we're passing the temporary file in a read-mode file descriptor to S3

180. By Jay Pipes on 2011-08-03

Removes the call to webob.Request.make_body_seekable() in the
general images controller to prevent the image from being copied
into memory. In the S3 controller, which needs a seekable file-like
object when calling boto.s3.Key.set_contents_from_file(), we work
around this by writing chunks of the request body to a tempfile on
the API node, then stream this tempfile to S3.

179. By Brian Waldon on 2011-08-03

- removed curl api functional tests
- moved httplib2 api functional tests to tests/functional/test_api.py

178. By Soren Hansen on 2011-08-03

Make tests a package under glance.

177. By Brian Waldon on 2011-08-02

Add filter support to bin/glance index and details calls

176. By Brian Waldon on 2011-08-02

Update registry db api to properly handle pagination through sorted results.

175. By Jay Pipes on 2011-08-02

Our code doesn't work with python-xattr 0.5.0, and that's the version installed in RH/Centos :( Andrey has updated the RPM config to specify 0.6.0, and this does the same to pip-requires.

174. By Justin Shepherd on 2011-08-02

Replaced occurances of |str(e)| with |"%s" % e|.

Except in bin/glance, where |unicode(e).split('\n')| makes more sense than |("%s" % e).split('\n')|

173. By Jay Pipes on 2011-08-02

First round of refactoring on stores.

Refactors stores to be stateful:

* Creates a new base Store class
* Adds code for new Store modules to "register" with Glance
  and ensure that when the Controller(s) are spun up, that
  the stateful store objects are all created properly and passed
  configuration options. This allows us to refactor a bunch of code
  out of the various store adapters that is constantly re-checking
  configuration option values during each call to get/add/delete, etc
* Removes the now-unnecessary repetitve option-checking code in the
  Swift, S3, and filesystem drivers

172. By Justin Shepherd on 2011-07-31

Use of "%default" in help string does not work, have to use "%(default)s". Per the 4th example http://docs.python.org/dev/library/argparse.html#prog