]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Add update to volume and snapshot controllers
authorClay Gerrard <clay.gerrard@gmail.com>
Mon, 27 Aug 2012 21:38:25 +0000 (21:38 +0000)
committerClay Gerrard <clay.gerrard@gmail.com>
Tue, 11 Sep 2012 18:24:09 +0000 (13:24 -0500)
commit04ca527aa394376b20e1fc09b1cdd1ca8a0a105a
treeb83a36131cd1285440a8c9f524b52a61bc1f9183
parent2ed3eeb13514ce502ce1df873970404084b50999
Add update to volume and snapshot controllers

Add update method to api.openstack.volume.volumes and snapshots
controllers.  Url mapping automatically routes a PUT on the resource to
the update method.  Implementation closely matches updates to server
resource for the openstack compute api.

e.g.

curl http://localhost:8776/v1/${PROJECT_ID}/volumes/${VOLUME_ID} \
    -H 'x-auth-token: ${ADMIN_AUTH_TOKEN}' \
    -H 'content-type: application/json' \
    -XPUT -d '{"volume": {"display_name": "new-name"}}'

Add volume.api.API.update_snapshot method to avoid making db calls
directly from the SnapshotsController.

Add tests for new methods.

Update return value of test_volume.VolumeTestCase._create_snapshot
to return the whole model like _create_volume

fixes lp bug #1042904

Change-Id: I887178f9b7834cc7afd54bba8fb902b630faf8c0
cinder/api/openstack/volume/snapshots.py
cinder/api/openstack/volume/volumes.py
cinder/tests/api/openstack/fakes.py
cinder/tests/api/openstack/volume/test_snapshots.py
cinder/tests/api/openstack/volume/test_volumes.py
cinder/tests/integrated/api/client.py
cinder/tests/integrated/test_volumes.py
cinder/tests/policy.json
cinder/tests/test_volume.py
cinder/volume/api.py