summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | | | | | Merge "Update scheduler rpc API version."Jenkins2012-09-183-5/+7
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | |_|/ / / / / / / / / / | |/| | | | | | | | | | |
| * | | | | | | | | | | | Update scheduler rpc API version.Russell Bryant2012-09-183-5/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit de09c1866b9138610914ddaaebb9b030884d1e28 updated the scheduler rpc API but did not account for rpc API versioning. This patch fixes it up. It updates the manager to reflect that it implements '2.1' and is now backwards compatible with the old method signature. The client side rpcapi now indicates that it requires '2.1' when sending the new image_id argument. Fix bug 1052670. Change-Id: Ide541f38d3d0129809da56baffe95bc3faab0569
* | | | | | | | | | | | | Merge "Add deserialization for multiple create and az"Jenkins2012-09-182-1/+38
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | |_|/ / / / / / / / / / | |/| | | | | | | | | | |
| * | | | | | | | | | | | Add deserialization for multiple create and azVishvananda Ishaya2012-09-182-1/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We were not deserializing xml values causing the multiple create extension and the availability zone extension to fail with xml. This adds deserialization for these attributes and adds tests Part of bug 1050997 Change-Id: Ic40ce58b0854717dd99dba8ed39c861e31e6bba4
* | | | | | | | | | | | | Merge "Make sure to return an empty subnet list for a network without sunbet."Jenkins2012-09-182-0/+43
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / |/| | | | | | | | | | | |
| * | | | | | | | | | | | Make sure to return an empty subnet list for a network without sunbet.Akihiro MOTOKI2012-09-182-0/+43
| | |_|_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1043827. This commit fixes a bug that when launching an instance with a quantum network without subnet allocate_for_instance() in network/quantumv2/api.py returns non-related subnet info as instance_nw_info. A port created on a network without subnet is associated with no subnet and subnet info in instance_nw_info should be an empty list. Change-Id: I2cfb78886ab10dd94eb795f5288b26b77f379998
* | | | | | | | | | | | Merge "Correct db migration 91"Jenkins2012-09-182-6/+210
|\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | |
| * | | | | | | | | | | Correct db migration 91Brian Waldon2012-09-182-6/+210
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Snapshot.id gets the new snapshot uuid rather than snapshot.volume_id * Foreign keys are dropped before and recreated after updating id fields * Snapshot id <-> uuid queries use snapshot_id_mappings.c.id rather than volume_id_mappings.id * Snapshot id <-> uuid queries are executed before passing the new id values into subsequent UPDATE queries * Thoroughly inspect the expected modifications in a new functional test * Fixes bug 1052244 * Fixes bug 1052220 Change-Id: I22c820e5747562251c6447ac678c80dd9e0e2e20
* | | | | | | | | | | | Merge "Fix doc/README.rst to render properly"Jenkins2012-09-181-8/+9
|\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | Fix doc/README.rst to render properlyJoe Gordon2012-09-181-8/+9
| | |_|/ / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix doc/README.rst to work on github: https://github.com/openstack/nova/tree/master/doc Change-Id: Ib589607014158c2a5af26bd137e252e5a0de30c6
* / | | | | | | | | | | Adds API sample testing for Extended server attributes extensionNikola Dipanov2012-09-1817-0/+458
|/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds the basic test and templates for testing the response of the extended server attributes extension. Tests both the json and XML responses. Change-Id: I38f43263bf5de854d7d7cf6d2180cf68831ad09b
* | | | | | | | | | | Merge "add ability to clone images"Jenkins2012-09-183-7/+260
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | add ability to clone imagesJosh Durgin2012-09-183-7/+260
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Given the backend location from Glance, drivers can determine whether they can clone or otherwise efficiently create a volume from the image without downloading all the data from Glance. For now implement cloning for the RBD driver. There's already a Glance backend that stores images as RBD snapshots, so they're ready to be cloned into volumes. Fall back to copying all the data if cloning is not possible. Implements: blueprint efficient-volumes-from-images Signed-off-by: Josh Durgin <josh.durgin@inktank.com> Conflicts: nova/volume/api.py nova/volume/driver.py This is based on a cherry-pick of cinder commit edc11101cbc06bdce95b10cfd00a4849f6c01b33 Change-Id: I71a8172bd22a5bbf64d4c68631630125fcc7fd34
* | | | | | | | | | | | Merge "add get_location method for images"Jenkins2012-09-183-27/+51
|\| | | | | | | | | | |
| * | | | | | | | | | | add get_location method for imagesJosh Durgin2012-09-183-27/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is useful for determining whether the backend storage for the image in Glance is the same as the storage used in a volume driver. The direct_url is only available in the v2 images API, so add a version parameter to each request. As more parts of the v2 API are used, this parameter will become more useful. Signed-off-by: Josh Durgin <josh.durgin@inktank.com> Conflicts: nova/image/glance.py This is based on a cherry-pick of cinder commit 171a57a6879683dac5f219a0e8f1080ea687105d Change-Id: I5ca06b6d80ffe4118f8dd381ed11283573bce71f
* | | | | | | | | | | | Merge "Adds new volume API extensions"Jenkins2012-09-1824-48/+916
|\| | | | | | | | | | | | |_|/ / / / / / / / / |/| | | | | | | | | |
| * | | | | | | | | | Adds new volume API extensionsUnmesh Gurjar2012-09-1824-48/+916
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds following extensions: 1. Create volume from image 2. Copy volume to image Added unit tests. Implements: blueprint create-volume-from-image Conflicts: cinder/api/openstack/volume/contrib/volume_actions.py cinder/tests/api/openstack/fakes.py cinder/tests/api/openstack/volume/contrib/test_volume_actions.py cinder/tests/policy.json nova/api/openstack/volume/volumes.py nova/flags.py nova/tests/api/openstack/volume/test_volumes.py nova/tests/test_volume.py nova/utils.py nova/volume/api.py nova/volume/manager.py This is based on a cherry-pick of cinder commit 2f5360753308eb8b10581fc3c026c1b66f42ebdc with bug fixes 8c30edff982042d2533a810709308b586267c0e9 and ffe5036fa0e63ccde2d19aa0f425ec43de338dd7 squashed in. Change-Id: I9c73bd3fa2fa2e0648c01ff3f4fc66f757d7bc3f
* | | | | | | | | | | Merge "Fix volume id conversion in nova-manage volume"Jenkins2012-09-181-6/+21
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Fix volume id conversion in nova-manage volumeJohn Griffith2012-09-171-6/+21
| | |_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The nova-manage volume commands weren't updated when the switch from int volume-id's to uuid's was made. This updates the param2id method to call the appropriate conversion, and doesn't attempt to cast the value to an int. There is an additional issue with the nova-manage commands with the introduction of Cinder. The calls in nova-manage are direct to the nova database which when using Cinder is not valid. Add an error message when trying to use delete or re-attach commands as they don't work. This change deprecates the nova-manage delete function as it's now implemented as an admin extension (nova commit id: If795599d and cinder commit id: I29f4b892). The reattach command is only valid for nova-volumes, a similar extension will need to be added to cover cinder at which time the nova-manage volume reattach can be deprecated as well. Partial fix for bug #1051603 Change-Id: I666d4e627dee4a2025f7135560ee36c40f4bf17a
* | | | | | | | | | | Merge "Add console output extension to API samples test."Jenkins2012-09-1817-0/+161
|\ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | Add console output extension to API samples test.Matthew Treinish2012-09-1817-0/+161
| | |_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I0ac1f893b06882acaa7c61dade61eb991a5d0d04 Signed-off-by: Matthew Treinish <treinish@linux.vnet.ibm.com>
* | | | | | | | | | | Merge "Fixes snat rules in complex networking configs"Jenkins2012-09-182-14/+20
|\ \ \ \ \ \ \ \ \ \ \ | |_|_|/ / / / / / / / |/| | | | | | | | | |
| * | | | | | | | | | Fixes snat rules in complex networking configsVishvananda Ishaya2012-09-102-14/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is currently no way to disable nova's snat rule, which causes problems for some network configurations with an external gateway. This patch allows the snat to be completely disabled by setting a blank value for routing_source_ip. This patch also makes the snat rule a little more specific, only snatting traffic that is destined for the public interface (or the floating interface if it is defined). This allows the snat to work without changing routing_source_ip and also ensures that an instance assigned multiple ips from different floating pools will connect from the appropriate address for each pool Fixes bug 1048765 Change-Id: I18be88a3dbb7a9f4762db7beefc94e07b8310100
* | | | | | | | | | | Merge "Fix volume deletion when device mapper is used"Jenkins2012-09-182-0/+8
|\ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / |/| | | | | | | | | |
| * | | | | | | | | | Fix volume deletion when device mapper is usedRafi Khardalian2012-09-172-0/+8
| | |_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Call dmsetup remove if there is a /dev/mapper/nova--volumes- element present. Resolves bug 979020 Change-Id: Iddaaed411a77dda4bd32f9a97687ff17744119eb
* | | | | | | | | | Merge "Make DeregisterImage respect AWS EC2 specification"Jenkins2012-09-182-2/+2
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / / |/| | | | | | | | |
| * | | | | | | | | Make DeregisterImage respect AWS EC2 specificationMotoKen2012-09-172-2/+2
| | |_|_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The return value of current deregister_image implementation is the AMI id. Howerver, according to Amazon EC2 API documentation, the return value of DeregisterImageResponse should be boolean. See more detailed documentation in: http://docs.amazonwebservices.com/AWSEC2/latest/APIReference/ApiReference-query-DeregisterImage.html The example response: <DeregisterImageResponse xmlns="http://ec2.amazonaws.com/doc/2012-08-15/"> <requestId>59dbff89-35bd-4eac-99ed-be587EXAMPLE</requestId> <return>true</return> </DeregisterImageResponse> This patch changes the response and test case for deregister_image. Fixes bug 1051752. Change-Id: Ifd31b88ec4f575f50244079cdeed4c9ff57206f8
* | | | | | | | | Merge "Fixes error handling during schedule_run_instance"Jenkins2012-09-188-82/+146
|\ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / |/| | | | / / / / | | |_|_|/ / / / | |/| | | | | |
| * | | | | | | Fixes error handling during schedule_run_instanceVishvananda Ishaya2012-09-178-82/+146
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If there are not enough hosts available during a multi-instance launch, every failing instance should be updated to error state, instead of just the first instance. Currently only the first instance is set to Error and the rest stay in building. This patch makes a number of fixes to error handling during scheduling. * Moves instance faults into compute utils so they can be created from the scheduler. * Moves error handling into the driver so that each instance can be updated separately. * Sets an instance fault for failed scheduling * Sets task state back to none if there is a scheduling failure * Modifies chance scheduler to stop returning a list of instances as it is not used. * Modifies tests to check for these states. In addition to the included tests, the code was manually verified on a devstack install Fixes bug 1051066 Fixes bug 1019017 Change-Id: I49267ce4a21e2f7cc7a996fb2ed5d625f6794730
* | | | | | | Merge "Allows waiting timers in libvirt to raise NotFound"Jenkins2012-09-182-58/+23
|\ \ \ \ \ \ \
| * | | | | | | Allows waiting timers in libvirt to raise NotFoundVishvananda Ishaya2012-09-172-58/+23
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are cases where an operation will fail when communicating with libvirt. We were eating the exception even though the operation failed, which has the potential to put the instance into an unrecoverable state. This patch allows NotFound exceptions to propogate up so that they are caught by the state handling code and the task state can be set to error. Fixes bug 1002814 Change-Id: Iddc319b24aee0b7132155f50b9d3b0eee9bb3fa8
* | | | | | | Merge "Add admin actions extension"Jenkins2012-09-1814-17/+398
|\ \ \ \ \ \ \
| * | | | | | | Add admin actions extensionClay Gerrard2012-09-1414-17/+398
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The optional os-admin-actions extension adds new wsgi_actions to the volumes/action resource and a new snapshots/action endpoint. With this extension both controllers will support an os-reset_status action to force a database update of a volume or snapshot that is stuck in a failed/incorrect status. The os-reset_status action works similarly to the compute api's os-reset_state action for instances. The os-force_delete action behaves similarly to the "cinder-manage volume delete" command and allows operators/admins to retry the delete operation after it has gone into an error_deleting status with an admin api call. The os-admin-actions extension is enabled by default, but limited to the admin api by the default policy.json rules. Individual admin actions can be disabled with policy rules as well. Example of os-reset_status action on a volume: curl http://localhost:8776/v1/${PROJECT_ID}/volumes/${VOLUME_ID}/action \ -H "x-auth-token: ${ADMIN_AUTH_TOKEN}" \ -H 'content-type: application/json' \ -d '{"os-reset_status": {"status": "error"}}' The new admin only api can assist deployers who encounter bugs or operational issues that result in failed actions. It can also be used by future storage backends to support async callback style status updates from long running actions or operations which have encountered an error will be retried. Also updates the api.openstack.wsgi.ControllerMetaclass to support sub-classing wsgi.Controllers that define wsgi_actions. Partial fix for bug #1039706 Change-Id: If795599d5150dea362279d75a75276f3166d0149
* | | | | | | | Merge "Include volume_metadata with object on vol create"Jenkins2012-09-182-2/+20
|\ \ \ \ \ \ \ \
| * | | | | | | | Include volume_metadata with object on vol createJohn Griffith2012-09-172-2/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix for Bug 1029762 The symptom of this bug is that the response data of an OSAPI create call always shows and empty dict for volume_metadata regardless of what was passed in to created and actually used. Upon the db create_volume call a reference to the volume object is all that was being returned. Since metadata is specified and set, it should also be returned with the create reference object. This will result in the the osapi create call returning a body with correct metdata info rather than always showing and empty dict as it was previously. Change-Id: I469e06941f446526bae39168f3b1ebfea851e0ef
* | | | | | | | | Merge "Add man pages"Jenkins2012-09-1824-3/+1053
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ / |/| | | | | | | |
| * | | | | | | | Add man pagesJoe Gordon2012-09-1724-3/+1053
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add partially written manpages for: * nova-all * nova-api-ec2 * nova-api-metadata * nova-api-os-compute * nova-api-os-volume * nova-api * nova-cert * nova-compute * nova-console * nova-consoleauth * nova-dhcpbridge * nova-network * nova-novncproxy * nova-objectstore * nova-rootwrap * nova-rpc-zmq-receiver * nova-scheduler * nova-volume-usage-audit * nova-volume * nova-xvpvncproxy Change-Id: I3734831ce2f6b5d765e98b3f50fe8c1ad7965685
* | | | | | | | | Merge "Reset the task state after backup done."Jenkins2012-09-171-1/+8
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Reset the task state after backup done.eddie-sheffield2012-09-171-1/+8
| | |_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The compute manager needs to reset the task state after a backup is complete. However it was only correctly reseting after a snapshot due to the current expected state being hardcoded. Now looks for the correct expected state depending on whether a backup or snapshot was being executed. Fixes bug 1051069 Change-Id: Ie91f7745b98232370be6c99d318a8033a93cc50c
* | | | | | | | | Merge "Includes sec group quota details in limits API response"Jenkins2012-09-176-4/+29
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Includes sec group quota details in limits API responseRam Nalluri2012-09-136-4/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added the security groups quota details to the limit API response so that these details also get displayed to the user. Fixes bug 1049578 Change-Id: I45c345141747f99359c8948e959030f8cb99e866
* | | | | | | | | | Merge "Add api samples to Scheduler hints extension"Jenkins2012-09-179-0/+108
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Add api samples to Scheduler hints extensionMauro S. M. Rodrigues2012-09-159-0/+108
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ic324d796dac0cfc7b36c47a181b55e09cb1503e2
* | | | | | | | | | | Merge "Include Schedule Hints deserialization to XML API"Jenkins2012-09-172-0/+23
|\| | | | | | | | | | | |_|_|_|/ / / / / / |/| | | | | | | | |
| * | | | | | | | | Include Schedule Hints deserialization to XML APIMauro S. M. Rodrigues2012-09-152-0/+23
| | |_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | So far the xml format to schedule hints were not defined. Once defined (in ML), this patch add support to its deserialization. This patch also includes a xml deserialization method to get the Elements between the childs of a node. Partially fixes bug 1050997 Change-Id: I2a34dbbd6200755818d7eaa7330a96d61a043614
* | | | | | | | | Improve entity validation in volumes APIsMark McLoughlin2012-09-1712-89/+242
| |_|/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug #1048565 Use the new Controller.is_valid_body() helper to validate the entity body in various volumes related POST/PUT handlers and return 422 as appropriate. (Cherry picks commit dcecb586 from Cinder and adds similar fixes for the volumes bits in the compute API) Change-Id: I04127972981522c1ed81903893396c4f9665bcd3
* | | | | | | | Merge "Deserialize user_data in xml servers request"Jenkins2012-09-172-7/+63
|\ \ \ \ \ \ \ \
| * | | | | | | | Deserialize user_data in xml servers requestVishvananda Ishaya2012-09-162-7/+63
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also adds better validation for b64 encoded data since python will happily ignore illegal bytes in the base64 encoded data. Fixes bug 1050797 Change-Id: I4f380d029ec2d51f130a6dbd581410873e414216
* | | | | | | | Merge "Clean up handling of project_only in network_get"Jenkins2012-09-176-76/+82
|\ \ \ \ \ \ \ \
| * | | | | | | | Clean up handling of project_only in network_getVishvananda Ishaya2012-09-146-76/+82
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There was some funky logic for getting networks to work around the project only decorator. This changes the code to match what we actually want which is: In Flat and FlatDHCP mode non-admins should be able to access networks that belong to their project or networks that have no project_id assigned. In VlanManager, project_id=None projects should not be accessible as this means the project hasn't been assigned yet. The assignment is done with an elevated context. This patch adds some logic to model_query to allow None in the project_only filter and makes network_get_all_by_uuids and network_get use it. fixes bug 1048869 Change-Id: I5377cea87dec8e9d0d9cec84e07128c5c6e8dca3