summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | Fix deallocate_fixed_ip invocationMate Lakat2012-08-313-1/+22
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes bug 1043905. The host parameter was not passed to deallocate_fixed_ip method, so that the deallocation tried to do a remote call to another host, which timed out, thus instance termination failed. Change-Id: I18d6de31d15767a45f389ae14769985c3951d560
* | | | | | | Merge "Makes sample testing handle out of order output"Jenkins2012-08-311-41/+80
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | |
| * | | | | | Makes sample testing handle out of order outputVishvananda Ishaya2012-08-311-41/+80
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Changes the output comparison to compare nested python objects. This assumes assumes that lists in the expected output will always sort in the same order as lists in the actual output. Change-Id: I0fb02beb9a278f6f93d9b0eaf6d55fcd2a3c96b7
* | | | | | | Merge "Adds api sample testing for versions"Jenkins2012-08-317-8/+73
|\ \ \ \ \ \ \
| * | | | | | | Adds api sample testing for versionsVishvananda Ishaya2012-08-287-8/+73
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch tests api samples for the versions resource in json and xml. It makes a small modification to the integrated api client to support making requests above /v2/tenant_id. This also fixes an uncovered bug in the link handling in the versions resource. Fixes bug 1041426 Change-Id: Ie898ef3822fa250def2e0fa407a0968275262260
* | | | | | | | Merge "Save the original base image ref for snapshots."Jenkins2012-08-312-3/+24
|\ \ \ \ \ \ \ \
| * | | | | | | | Save the original base image ref for snapshots.Rick Harris2012-08-312-3/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Save the base image ref in instance system metadata for tracking purposes. (This is modified code originally from Rick Harris.) Change-Id: I574c597c7ff2e55675ebdaa7b3213b53e61a94b7
* | | | | | | | | Merge "Remove scheduler RPC API version 1.x"Jenkins2012-08-3110-664/+86
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Remove scheduler RPC API version 1.xMark McLoughlin2012-08-2910-664/+86
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Like with the compute RPC API, we're unlikely to still work well with RPC client using older 1.x version because of DB schema changes. In that case, we may as well remove 1.x support in Folsom and rip out the potentially buggy backwards compat code. This should also make backporting fixes from Grizzly easier. Deployers following trunk can upgrade all their nodes to use the version 2.0 API before deploying this commit. Change-Id: Iee099751bda9637da5e134357d28e89d5fba9895
* | | | | | | | | | Merge "Add version 2.0 of the scheduler RPC API"Jenkins2012-08-313-14/+85
|\| | | | | | | | |
| * | | | | | | | | Add version 2.0 of the scheduler RPC APIMark McLoughlin2012-08-293-14/+85
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Like we did for the compute RPC API, add version 2.0 of the scheduler API while still retaining 1.x support. This allows all services to be updated with 2.0 support before 1.x support is removed. Change-Id: If438d5b6f521ea4c487d1490d4b8cf6547ba5c72
* | | | | | | | | | Merge "Remove some remnants of VSA support"Jenkins2012-08-312-31/+0
|\| | | | | | | | |
| * | | | | | | | | Remove some remnants of VSA supportMark McLoughlin2012-08-292-31/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The VSA code removed by commit a3bab242 was the only user and implementor of the 'create_volumes' RPC method. Change-Id: Ic718d54d938059b034b78bc287668b0b9fa27f94
* | | | | | | | | | Merge "Fixed boot from snapshot failure."Jenkins2012-08-311-1/+5
|\ \ \ \ \ \ \ \ \ \ | |_|_|_|/ / / / / / |/| | | | | | | | |
| * | | | | | | | | Fixed boot from snapshot failure.Unmesh Gurjar2012-08-311-1/+5
| | |_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Passed the snapshot_id to cinder client from create API. Previously it was failing since snapshot was being passed instead of snapshot_id. Fixes LP: #1029211 Change-Id: I4aa1aca9067b2a282b9b46d909ef8342bf933a46
* | | | | | | | | Merge "Clean up non-spec output in flavor extensions"Jenkins2012-08-3117-101/+294
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Clean up non-spec output in flavor extensionsVishvananda Ishaya2012-08-2917-101/+294
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds option to cache flavors in the request object like instances. Modifies the flavorextradata extension to use the cache and optimizes tests. Adds flavor_disabled extension to control the extra data and includes tests. Fixes api samples to show the new extension. Fixes bug 1043585 Change-Id: Ie89df24a2891e3869d3fb604e07c79e8c913f290
* | | | | | | | | | Merge "Adds api sample testing for extensions endpoint"Jenkins2012-08-316-9/+943
|\| | | | | | | | |
| * | | | | | | | | Adds api sample testing for extensions endpointVishvananda Ishaya2012-08-296-9/+943
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Includes minor updates to base class to handle multiline descriptions and makes the extensions endpoint use a sorted list of extensions. Change-Id: I3604e3969b892646d5bbe08af44d454ee40e6f9e
* | | | | | | | | | Merge "Stop using scheduler RPC API magic"Jenkins2012-08-3111-20/+84
|\ \ \ \ \ \ \ \ \ \ | | |_|/ / / / / / / | |/| | | | | | | |
| * | | | | | | | | Stop using scheduler RPC API magicMark McLoughlin2012-08-2911-20/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a scheduler RPC message isn't handled directly by a SchedulerManager method, the __getattr__() fallback passes the message to a driver method in the form of schedule_${method}() and, if that doesn't exist, instead calls the schedule() method supplying the topic and method args. This is pretty bizarre stuff and we appear to only use it in two cases: 1) live_migration - this is how the schedule_live_migration() method in the driver gets called, but the side-effect is that we require the client to pass a topic argument which is never used. This would be much more sanely handled with an explicit SchedulerManager.live_migration() method. 2) create_volume - the volume API asks the scheduler to pick a target host for create_volume() using this method. This would be easily handled with an SchedulerManager.create_volume() method. Change-Id: I1047489d85ac51d8d36fea1c4eb858df638ce349
* | | | | | | | | | Merge "Remove compute RPC API version 1.x"Jenkins2012-08-314-876/+187
|\| | | | | | | | |
| * | | | | | | | | Remove compute RPC API version 1.xMark McLoughlin2012-08-284-876/+187
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now that we have RPC API versions 1.x and 2.0, we could choose to continue to support version 1.x in Folsom and only remove it in Grizzly. This, in theory, would allow smoother upgrades of older Folsom versions to the final Folsom version. However, in reality, we most likely made DB schema changes which affect the compute service and would prevent such a smooth upgrade anyway. Removing 1.x support now removes a lot of potentially buggy backwards compat code from the release and will also make backporting fixes from Grizzly to Folsom easier. Change-Id: Ic18017a16c5bffee85a43db65ff17283599a27ba
* | | | | | | | | | Merge "Add version 2.0 of compute RPC API"Jenkins2012-08-316-170/+520
|\| | | | | | | | | | |_|_|/ / / / / / |/| | | | | | | |
| * | | | | | | | Add version 2.0 of compute RPC APIMark McLoughlin2012-08-286-170/+520
| | |_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | RPC API versioning is helpful because (a) it gives useful error messages if there is a version mismatch and (b) it allows you to update to a new version of the API on the server side before updating the client side. However, we still regularly encounter DB changes which force the client and server side to be updated together. This will not be resolved until no-db-compute is fully implemented. It's safe to say that RPC API clients using version 1.0 will not work with a version 1.44 compute manager (because of DB schema changes). The issue, then, is that continuing to support 1.0 clients in the Folsom release seems pointless and the alternative of removing legacy RPC API support would mean a lot less cruft code in the release. Before removing version 1.x of the compute RPC API, we add version 2.0 while still retaining 1.x support. This gives users the option of upgrading all compute RPC API users to use version 2.0 before version 1.x support is removed from compute nodes. Change-Id: I1c74b36a84d770b0af5ca33062e069a545537ee2
* | | | | | | | Merge "FLAG rename: bandwith_poll_*=>bandwidth_poll_*"Jenkins2012-08-313-3/+5
|\ \ \ \ \ \ \ \
| * | | | | | | | FLAG rename: bandwith_poll_*=>bandwidth_poll_*Joe Gordon2012-08-303-3/+5
| | |_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix bug 1036407 bandwith_poll_interval is deprecated in Folsom and will be removed in Grizzly Change-Id: I32cbdffce21a0024bd0303d46d7b7632db6ec8cb
* | | | | | | | Update zmq context cleanup to use term.Dan Prince2012-08-311-1/+1
| |_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updates the RPC cleanup function for the zmq RPC driver to use Context.term() instead of Context.destroy(). This allows the Nova code to work with older versions (pre 2.1.10) of pyzmq. The Context term() function seems to do the job just as well as destroy() (and is also thread safe) so lets just use it instead. Fixes LP Bug #1044004. Change-Id: I07a200d4b216c0c1e155b57588295b35a237576c
* | | | | | | Merge "Makes api extension names consistent"Jenkins2012-08-317-10/+11
|\ \ \ \ \ \ \ | |/ / / / / / |/| | / / / / | | |/ / / / | |/| | | |
| * | | | | Makes api extension names consistentVishvananda Ishaya2012-08-297-10/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We have a few extensions that are using strange _ in their names. The Class names have to be named this way to be picked up properly by the loading code, but the user facing name is much nicer in PascalCase, so this switches them over. Change-Id: I9326410add1317ef006559d5ca2205a4487371e2
* | | | | | Merge "Spelling: Persistant=>Persistent"Jenkins2012-08-301-1/+1
|\ \ \ \ \ \
| * | | | | | Spelling: Persistant=>PersistentJoe Gordon2012-08-301-1/+1
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | Change-Id: I0d22e3f03e8f1a74a8bce7c76c93c9c7170defd5
* | | | | | Merge "External locking for image caching."Jenkins2012-08-304-19/+45
|\ \ \ \ \ \
| * | | | | | External locking for image caching.Michael Still2012-08-294-19/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the instance storage is shared between compute nodes, then you need external locking which is also shared to avoid clobbering each other's attempts to cache base images. Resolves bug 1014227. Change-Id: Ic2ac87840904fa199c17774dae9556ad6c7a3eaf
* | | | | | | Merge "delete unused valiables"Jenkins2012-08-301-2/+0
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | |
| * | | | | | delete unused valiablesjiangwt1002012-08-301-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I65f5ca04882a0738eaad1f61e255d371fa6204ae
* | | | | | | Merge "Assume virt disk size is consumed by instances."Jenkins2012-08-303-11/+14
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | Assume virt disk size is consumed by instances.Yaguang Tang2012-08-293-11/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use disk_available_least other than free_disk_gb to consume. assume virt disk size is consumed by instances when calculate the available disk size. fix bug lp:1042071 Change-Id: I82dc3bcfd97219085c624c81dfa47cbf0bcd4504
* | | | | | | Merge "Add a new exception for live migration."Jenkins2012-08-303-5/+9
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | |
| * | | | | | Add a new exception for live migration.Jian Wen2012-08-293-5/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Raises new exception InvalidLocalStorage instead of InvalidSharedStorage when block migration is used with shared storage. Corrects the message of InvalidSharedStorage. Fixes bug 1043165 Change-Id: I68bfac3f89276edc38c00ca963911fe07456dc0d
* | | | | | | Merge "Spelling fix Retrive=> Retrieve"Jenkins2012-08-301-2/+2
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | |
| * | | | | | Spelling fix Retrive=> RetrieveJoe Gordon2012-08-291-2/+2
| |/ / / / / | | | | | | | | | | | | | | | | | | Change-Id: I7eeb8b8477884b1b4e847fe3d76b9b7d578bda8a
* | | | | | Merge "Implements PowerVM get_available_resource method."Jenkins2012-08-303-11/+43
|\ \ \ \ \ \
| * | | | | | Implements PowerVM get_available_resource method.Tiago Mello2012-08-293-11/+43
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implements get_available_resource method in the PowerVM driver to fix the failure when launching nova-compute. Also, it removes the update_available_resource method from PowerVMDriver which is no longer required. Fixes bug 1040789 Change-Id: Ibc7e9b80e6d9b8ef3c1a844783a890c592517f89
* | | | | | Merge "hacking: Add driver prefix recommendation."Jenkins2012-08-301-0/+4
|\ \ \ \ \ \
| * | | | | | hacking: Add driver prefix recommendation.Rick Harris2012-08-291-0/+4
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Prefixing the commit message's first line with the driver makes it easier for reviewers, at a glance, to find reviews that relate to their area of expertise as well as improving the overall readability of the git history. Change-Id: I376d2cdb5dc344717fb7749a80e33ee88603e68c
* | | | | | Merge "Sort API extensions by alias."Jenkins2012-08-306-14/+20
|\ \ \ \ \ \
| * | | | | | Sort API extensions by alias.Dan Prince2012-08-296-14/+20
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updates the ExtensionManager so that extensions are sorted by alias when accessed via the get_resources or get_controller_extensions functions. This fixes the integrated.test_api_samples.ServersSampleAllExtensionXmlTest.test_servers_get test which fails on Fedora 17 (but passes on Fedora 16 and apparently some other distros as well). Includes updated api_samples for XML that was regenerated by the test_servers_get test. Fixes LP Bug #1042709. Change-Id: I82c679f457a3cd11c8a8cd76f402693b61bb14cc
* / / / / / Update requires to glanceclient >=0.5.0Vishvananda Ishaya2012-08-291-1/+1
|/ / / / / | | | | | | | | | | | | | | | Change-Id: If788b94a142f469b5ce47c49a4dcdb05f1455cce
* | | | | Merge "Do not run pylint by default"Jenkins2012-08-291-1/+1
|\ \ \ \ \ | |/ / / / |/| | | |